On Tue, Nov 24, 2015 at 1:06 AM, Amit Langote <langote_amit...@lab.ntt.co.jp> 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 make it into the tree. Attached fixes it. Also, wondering if > the semantics of Gather node is that of Scan or more generic Plan? That is > to ask whether the following edit makes sense: > > * nodeGather.c > - * Support routines for scanning a plan via multiple workers. > + * Support routines for getting the result from a plan via multiple > + * workers. > *
Well I think "scanning a plan" is clear enough even if it's technically a Scan. But I agree the second change is needed. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
