Hello Kyotaro-san,

If you feel that this feature only deserve a lexer solution, then the
patch should be "returned with feedback".

It's unfortunate to abandon this idea so I tried this and made it run with psql's parser. I think it works as expected.

Wow, you are much more courageous than I am!:-)

- 0001-Prepare-for-share-psqlscan-with-pgbench.patch
A patch to modify psql so that psqlscan can be shared with other modules.

- 0002-Make-use-of-psqlscan-in-pgbench.patch
A patch to use psqlscan in pgbench.

- hoge.sql
A sample custom script including multilne statement and line comment

I can't judge wheter this is a new version of Febien's patch
following Tom's suggestion or brand-new one. Anyway I'd like to
post on this thread.

I think it is really a new patch, but posting it is seems logical because that is where the discussion was lead.

- 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:-)

Suggestions? Opinions?

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.

- Modifying psqlscan in psql requires consideration on how it is
 used in pgbench.

Yep, that is one of the reason why I did not want to go this way, bar my natural lazyness.

--
Fabien.


--
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