Re: [PATCHES] Minor Comment updates

2005-04-24 Thread Tom Lane
Neil Conway <[EMAIL PROTECTED]> writes: > Simon Riggs wrote: >> These are the only two files in /executor that have old QUEL comments in >> to describe their behaviour, AFAICS. > Applied with some minor cleanups. Thanks for the patch. I restored the one example Simon removed, as I think it's a us

Re: [PATCHES] Minor Comment updates

2005-04-24 Thread Neil Conway
Simon Riggs wrote: These are the only two files in /executor that have old QUEL comments in to describe their behaviour, AFAICS. Applied with some minor cleanups. Thanks for the patch. -Neil ---(end of broadcast)--- TIP 6: Have you searched our list a

Re: [PATCHES] Minor Comment updates

2005-04-22 Thread Simon Riggs
On Thu, 2005-04-21 at 23:52 -0400, Alvaro Herrera wrote: > On Thu, Apr 21, 2005 at 11:15:48PM +0100, Simon Riggs wrote: > This one is wrong ... it really means EXPLAIN EXECUTE. > An updated patch, with changes to nodeAppend.c and nodeResult.c (only) These are the only two files in /executor tha

Re: [PATCHES] Minor Comment updates

2005-04-22 Thread Simon Riggs
On Thu, 2005-04-21 at 23:52 -0400, Alvaro Herrera wrote: > On Thu, Apr 21, 2005 at 11:15:48PM +0100, Simon Riggs wrote: > > > This one is wrong ... it really means EXPLAIN EXECUTE. > OK, yes that one is wrong - thought it was old syntax. Best Regards, Simon Riggs --

Re: [PATCHES] Minor Comment updates

2005-04-21 Thread Alvaro Herrera
On Thu, Apr 21, 2005 at 11:15:48PM +0100, Simon Riggs wrote: This one is wrong ... it really means EXPLAIN EXECUTE. > Index: explain.c > === > RCS file: /projects/cvsroot/pgsql/src/backend/commands/explain.c,v > retrieving revision