Re: pgbench - allow backslash-continuations in custom scripts

From: Josh Berkus <josh(at)agliodbs(dot)com>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Cc: PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: pgbench - allow backslash-continuations in custom scripts
Date: 2015-06-21 20:22:28
Message-ID: 55871D04.8070005@agliodbs.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Fabien,

> Without a lexer it is possible to fool pgbench with somehow contrived
> examples, say with:
>
> SELECT 'hello;
> world';
>
> The ";" within the string will be considered as end-of-line.
>
> Also, comments intermixed with sql on the same line would generate errors.
>
> SELECT 1 -- one
> + 3;
>
> Would fail, but comments on lines of their own are ok.
>
> It may be argued that these are not a likely scripts and that this
> behavior could be declared as a "feature" for keeping the code simple.

Yeah, these seem pretty contrived. I would personally be OK with
breaking them.

> ISTM that it would be an overall improvement, but also the ;-termination
> requirement breaks backward compatibility.

Look, how many people in the world develop their own pgbench scripts?
And how many of those aren't on this list right now, reading this
thread? I expect I could count them on my fingers and toes.
Backwards-compatability for pgdump, pg_basebackup, initdb, etc. matters.
The worst case with pgbench is that we break two people's test scripts,
they read the release notes, and fix them.

--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Fabien COELHO 2015-06-21 20:37:14 Re: pgbench - allow backslash-continuations in custom scripts
Previous Message Andres Freund 2015-06-21 19:39:58 Re: castoroides spinlock failure on test_shm_mq