From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Ian Lance Taylor <ian(at)airs(dot)com> |
Cc: | Vince Vielhaber <vev(at)michvhf(dot)com>, Justin Clift <justin(at)postgresql(dot)org>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Re: [PATCHES] Select parser at runtime |
Date: | 2001-08-13 13:57:17 |
Message-ID: | 16645.997711037@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Ian Lance Taylor <ian(at)airs(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2001-08-13 14:05:02 | Re: PL/pgSQL bug? |
Previous Message | Jan Wieck | 2001-08-13 13:55:59 | Re: Re: [PATCHES] Select parser at runtime |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2001-08-13 14:14:14 | Re: Patch: use SCM_CREDS authentication over PF_LOCAL sockets |
Previous Message | Jan Wieck | 2001-08-13 13:55:59 | Re: Re: [PATCHES] Select parser at runtime |