On 07/24/2015 11:36 AM, Kyotaro HORIGUCHI wrote:
At Fri, 24 Jul 2015 07:39:16 +0200 (CEST), Fabien COELHO <coe...@cri.ensmp.fr> wrote 
in <alpine.DEB.2.10.1507240731050.12839@sto>
- backslash commands is handled as the same as before: multiline
  is not allowed.

Hmm... that is really the feature I wanted to add initially, too bad
it is the dropped one:-)

Ouch. The story has been derailed somewhere.

Since SQL statments could be multilined without particluar
marker, we cannot implement multilined backslash commands in the
same way..

I don't think we actually want backslash-continuations. The feature we want is "allow SQL statements span multiple lines", and using the psql lexer solves that. We don't need the backslash-continuations when we have that.

On 07/25/2015 05:53 PM, Fabien COELHO wrote:
I don't have idea how to deal with the copy of psqlscan.[lh] from
psql. Currently they are simply the dead copies of those of psql.

I think that there should be no copies, but it should use relative
symbolic links so that the files are kept synchronized.

Yeah, I think so but symlinks could harm on git and Windows.
The another way would be make copies it from psql directory. They live
next door to each other.

Indeed there are plenty of links already which are generated by makefiles
(see src/bin/pg_xlogdump/*), and probably a copy is made on windows. There
should no file duplication within the source tree.

Yeah, following the example of pg_xlogdump and others is the way to go.

Docs need updating, and there's probably some cleanup to do before this is ready for committing, but overall I think this is definitely the right direction.

I complained upthread that this makes it impossible to use "multi-statements" in pgbench, as they would be split into separate statements, but looking at psqlscan.l there is actually a syntax for that in psql already. You escape the semicolon as \;, e.g. "SELECT 1 \; SELECT 2;", and then both queries will be sent to the server as one. So even that's OK.

- Heikki


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to