From: | Kouhei Kaigai <kaigai(at)ak(dot)jp(dot)nec(dot)com> |
---|---|
To: | Kouhei Kaigai <kaigai(at)ak(dot)jp(dot)nec(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Shigeru Hanada <shigeru(dot)hanada(at)gmail(dot)com> |
Subject: | Re: Foreign join pushdown vs EvalPlanQual |
Date: | 2015-11-18 01:47:23 |
Message-ID: | 9A28C8860F777E439AA12E8AEA7694F80117298C@BPXM15GP.gisp.nec.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> > On Sun, Nov 8, 2015 at 7:26 PM, Kouhei Kaigai <kaigai(at)ak(dot)jp(dot)nec(dot)com> wrote:
> > > The attached patch is an adjusted version of the previous one.
> > > Even though it co-exists a new callback and fdw_recheck_quals,
> > > the callback is kicked first as follows.
> >
> > This seems excessive to me: why would we need an arbitrary-length list
> > of plans for an FDW? I think we should just allow an outer child and
> > an inner child, which is probably one more than we'll ever need in
> > practice.
> >
> It just intends to keep code symmetry with custom-scan case, so not
> a significant reason.
> And, I expected ForeignScan will also need multiple sub-plans soon
> to support more intelligent push-down like:
> http://www.postgresql.org/message-id/9A28C8860F777E439AA12E8AEA7694F8010F47D
> A(at)BPXM15GP(dot)gisp(dot)nec(dot)co(dot)jp
>
> It is a separate discussion, of course, so I don't have strong preference
> here.
>
> > This looks like an independent bug fix:
> >
> > + fscan->fdw_recheck_quals = (List *)
> > + fix_upper_expr(root,
> > + (Node *)
> > fscan->fdw_recheck_quals,
> > + itlist,
> > + INDEX_VAR,
> > + rtoffset);
> > pfree(itlist);
> >
> > If so, it should be committed separately and back-patched to 9.5.
> >
> OK, I'll split the patch into two.
>
The attached patch is the portion cut from the previous EPQ recheck
patch.
Regarding of the fdw_plans or fdw_plan, I'll follow your suggestion.
Thanks,
--
NEC Business Creation Division / PG-Strom Project
KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
Attachment | Content-Type | Size |
---|---|---|
pgsql-bugfix-fdw_recheck_quals-on-setrefs.patch | application/octet-stream | 701 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Etsuro Fujita | 2015-11-18 02:29:04 | Re: Minor comment improvement to create_foreignscan_plan |
Previous Message | Masahiko Sawada | 2015-11-18 00:44:48 | Re: Freeze avoidance of very large table. |