Re: Re: [PATCHES] Select parser at runtime - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Re: [PATCHES] Select parser at runtime
Date
Msg-id 16645.997711037@sss.pgh.pa.us
Whole thread Raw
In response to Re: Re: [PATCHES] Select parser at runtime  (Ian Lance Taylor <ian@airs.com>)
Responses Re: Re: [PATCHES] Select parser at runtime
List pgsql-hackers
Ian Lance Taylor <ian@airs.com> writes:
> ... most of the cost will be paid by the people who care about
> it.  (Not all of the cost, because some communication will be required
> when the parse tree nodes are changed.)

> Along these lines, I don't think Bruce's suggestion of modifications
> to the Postgres gram.y is a good idea, because it causes the Oracle
> parser to add an ongoing cost to the Postgres parser.

And managing grammar changes and parse-tree-node changes is not an
ongoing cost?  I beg to differ.  We do that a lot, and keeping multiple
grammar files in sync is not a pleasant prospect.  (Look at ecpg ---
it's a major pain to keep it in sync with the main parser, even though
it only shares productions and not output code.  Worse, I have zero
confidence that it actually *is* in sync.)

If the grammar changes are small and localized, I think Bruce's #ifdef
approach might well be the way to go.  However, we're speculating in
a vacuum here, not having seen the details of the changes needed.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: PL/pgSQL bug?
Next
From: Jan Wieck
Date:
Subject: Re: PL/pgSQL bug?