> -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of Robert Haas > Sent: Wednesday, July 22, 2015 11:19 PM > To: Kaigai Kouhei(海外 浩平) > Cc: Etsuro Fujita; [email protected] > Subject: Re: [HACKERS] fdw_scan_tlist for foreign table scans breaks EPQ > testing, > doesn't it? > > On Wed, Jul 22, 2015 at 8:13 AM, Kouhei Kaigai <[email protected]> wrote: > >> No, what I'm concerned about is the case when scanrelid > 0. > >> > > Hmm. if scanrelid > 0, then fdw_scan_tlist should be NIL. > > I want to put Assert(scanrelid==0 || fdw_scan_tlist == NIL) just after > > the GetForeignPlan() in createplan.c. > > > > I'm curious why you tried to put valid fdw_scan_tlist for scanrelid > 0. > > It's unusual. > > Allowing that was part of the point of Tom Lane's commit > 1a8a4e5cde2b7755e11bde2ea7897bd650622d3e. See the second bullet > point, after the comma. > Indeed, this commit allows ForeignScan to have fdw_scan_tlist, even if scanrelid > 0, however, I'm uncertain about its reason/intention. Does it a preparation for the upcoming target-list-pushdown??
Thanks, -- NEC Business Creation Division / PG-Strom Project KaiGai Kohei <[email protected]> -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
