Re: pgbench - allow backslash-continuations in custom scripts

From: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgbench - allow backslash-continuations in custom scripts
Date: 2015-12-24 06:40:19
Message-ID: alpine.DEB.2.10.1512240729160.17411@sto
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Hello Michaël,

>> If I read you correctly, I should cut it out into a new file and
>> include it. Is it correct?
>
> Not really, I meant to see if it would be possible to include this set
> of routines directly in libpqcommon (as part of OBJS_FRONTEND). This
> way any client applications could easily reuse it. If we found that
> what was in psql is now useful to pgbench, I have little doubt that
> some other folks would make good use of that. I honestly have not
> looked at the code to see if that's doable or not, but soft-linking
> directly in pgbench a file of psql will not help future maintenance
> for sure. This increases the complexity of the code.

Just my 0.02€:

I understand that you suggest some kind of dynamic parametrization... this
is harder to do and potentially as fragile as the link/ifdef option, with
an undertermined set of callbacks to provide... the generic version would
be harder to debug, and this approach would prevent changing lexer
options. Basically I'm not sure that doing all that for improving the
handling of pgbench scripts is worth the effort. I would go with the
simpler option.

--
Fabien.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Langote 2015-12-24 06:59:21 Re: Additional role attributes && superuser review
Previous Message Fabien COELHO 2015-12-24 06:28:12 Re: Let PostgreSQL's On Schedule checkpoint write buffer smooth spread cycle by tuning IsCheckpointOnSchedule?