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

[PATCHES] Minor Comment updates

2005-04-21 Thread Simon Riggs
Some minor updates to comments. ? nodeResult.patch Index: nodeResult.c === RCS file: /projects/cvsroot/pgsql/src/backend/executor/nodeResult.c,v retrieving revision 1.29 diff -d -c -r1.29 nodeResult.c *** nodeResult.c 31 Dec 2004 21:59