Re: pgbench - refactor init functions with buffers
Fabien COELHO <coelho@cri.ensmp.fr>
From: Fabien COELHO <coelho@cri.ensmp.fr>
To: Andres Freund <andres@anarazel.de>
Cc: Jeevan Ladhe <jeevan.ladhe@enterprisedb.com>,
Dilip Kumar <dilipbalaut@gmail.com>,
PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2019-11-06T05:48:14Z
Lists: pgsql-hackers
Attachments
- pgbench-buffer-4.patch (text/x-diff) patch
Hello Andres, >> Attached v3 shorten some lines and adds "append_tablespace". A v4 which just extends the patch to newly added 'G'. > I'd prefer not to expand the use of pqexpbuffer in more places, and > instead rather see this use StringInfo, now that's also available to > frontend programs. Franckly, one or the other does not matter much to me. However, pgbench already uses PQExpBuffer, it uses PsqlScanState which also uses PQExpBuffer, and it intrinsically depends on libpq which provides PQExpBuffer: ISTM that it makes sense to keep going there, unless PQExpBuffer support is to be dropped. Switching all usages would involve a significant effort and having both PQExpBuffer and string_info used in the same file for the same purpose would be confusing. -- Fabien.
Commits
-
pgbench: Use PQExpBuffer to simplify code that constructs SQL.
- 9796f455c38e 14.0 landed
-
Make command order in test more sensible
- ad4b7aeb8443 13.0 cited