Re: Parser extensions (maybe for 10?) - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: Parser extensions (maybe for 10?)
Date
Msg-id CAFj8pRBujMChzXsKAq3xTpSB+g22P1GiPMqTyhRWXFH8Jd6Zew@mail.gmail.com
Whole thread Raw
In response to Re: Parser extensions (maybe for 10?)  (Simon Riggs <simon@2ndQuadrant.com>)
List pgsql-hackers


2016-04-19 12:49 GMT+02:00 Simon Riggs <simon@2ndquadrant.com>:
On 12 April 2016 at 06:51, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Craig Ringer <craig@2ndquadrant.com> writes:
> The other area where there's room for extension without throwing out the
> whole thing and rebuilding is handling of new top-level statements. We can
> probably dispatch the statement text to a sub-parser provided by an
> extension that registers interest in that statement name when we attempt to
> parse it and fail. Even then I'm pretty sure it won't be possible to do so
> while still allowing multi-statements. I wish we didn't support
> multi-statements, but we're fairly stuck with them.

Well, as I said, I've been there and done that.  Things get sticky
when you notice that those "new top-level statements" would like to
contain sub-clauses (e.g. arithmetic expressions) that should be defined
by the core grammar.  And maybe the extension would also like to
define additions to the expression grammar, requiring a recursive
callback into the extension.  It gets very messy very fast.

As Tom says, we can't easily break it down into multiple co-operating pieces, so lets forget that as unworkable.

What is possible is a whole new grammar... for example if we imagine

 SET client_language_path = 'foo, postgresql'

Works similar to search_path, but not userset. We try to parse incoming statements against the foo parser first, if that fails we try postgresql.
The default setting would be simply 'postgresql', so no match -> syntax error.

The idea is good. I don't understand to name "client_language_path" - it is not clean - a) this is server side feature, b) we use term "language" for PL, so any other term will be better.
 

We could make that easier by making the postgresql parser a plugin itself. So to produce a new one you just copy the files, modify them as needed then insert a new record into pg_language as an extension.

--
Simon Riggs                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: snapshot too old, configured by time
Next
From: Alvaro Herrera
Date:
Subject: Re: Reducing the size of BufferTag & remodeling forks