Re: [HACKERS] Minor comment edits in nodeGather.c

2015-11-30 Thread Amit Langote
On 2015/12/01 3:06, Robert Haas wrote: > On Tue, Nov 24, 2015 at 9:43 PM, Amit Langote > wrote: >> On 2015/11/25 11:31, Robert Haas wrote: >>> >>> Well I think "scanning a plan" is clear enough even if it's >>> technically a Scan. >> >> Okay, ripped that out in the attached. > > Committed, thanks

Re: [HACKERS] Minor comment edits in nodeGather.c

2015-11-30 Thread Robert Haas
On Tue, Nov 24, 2015 at 9:43 PM, Amit Langote wrote: > On 2015/11/25 11:31, Robert Haas wrote: >> On Tue, Nov 24, 2015 at 1:06 AM, Amit Langote >> wrote: >>> While going through nodeGather.c, I noticed portions of the file header >>> comment that may have been obsoleted by recent revisions of the

Re: [HACKERS] Minor comment edits in nodeGather.c

2015-11-24 Thread Amit Langote
On 2015/11/25 11:31, Robert Haas wrote: > On Tue, Nov 24, 2015 at 1:06 AM, Amit Langote > wrote: >> While going through nodeGather.c, I noticed portions of the file header >> comment that may have been obsoleted by recent revisions of the relevant >> parellelism code. For example, there is a refer

Re: [HACKERS] Minor comment edits in nodeGather.c

2015-11-24 Thread Robert Haas
On Tue, Nov 24, 2015 at 9:31 PM, Robert Haas wrote: > On Tue, Nov 24, 2015 at 1:06 AM, Amit Langote > wrote: >> While going through nodeGather.c, I noticed portions of the file header >> comment that may have been obsoleted by recent revisions of the relevant >> parellelism code. For example, the

Re: [HACKERS] Minor comment edits in nodeGather.c

2015-11-24 Thread Robert Haas
On Tue, Nov 24, 2015 at 1:06 AM, Amit Langote wrote: > While going through nodeGather.c, I noticed portions of the file header > comment that may have been obsoleted by recent revisions of the relevant > parellelism code. For example, there is a reference to PartialSeqScan node > which did not mak

[HACKERS] Minor comment edits in nodeGather.c

2015-11-23 Thread Amit Langote
While going through nodeGather.c, I noticed portions of the file header comment that may have been obsoleted by recent revisions of the relevant parellelism code. For example, there is a reference to PartialSeqScan node which did not make it into the tree. Attached fixes it. Also, wondering if the