Re: pgbench - refactor init functions with buffers

On 11/6/19 12:48 AM, Fabien COELHO wrote:
> 
> 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.

FWIW, I agree with Andres with regard to using StringInfo.

Also, the changes to executeStatementExpect() and adding 
executeStatement() do not seem to fit in with the purpose of this patch.

Regards,
-- 
-David
david@pgmasters.net