On Sat, Sep 26, 2015 at 5:52 AM, Robert Haas <
robertmhaas@gmail.com> wrote:
>
> On Fri, Sep 25, 2015 at 12:00 AM, Amit Kapila <
amit.kapila16@gmail.com> wrote:
> > I think initPlan will work with the existing patches as we are always
> > executing it in master and then sending the result to workers. Refer
> > below code in funnel patch:
>
> Sure, *if* that's what we're doing, then it will work. But if an
> initPlan actually attaches below a funnel, then it will break.
>
Currently, it's considered for initPlan of only left tree of Funnel, however
if we want to push multiple nodes under Funnel, then it won't work as
it is. I think even if we want to make that work, we would need to
traverse the whole tree under Funnel and do what currently is done for
each of the initPlan we encounter. In general, I think the idea of
passing the results of initPlan to workers seems like the right way
of dealing with initPlans and by the way this was a suggestion made
by you sometime back.