On Sat, Sep 26, 2015 at 5:52 AM, Robert Haas <robertmh...@gmail.com> wrote: > > On Fri, Sep 25, 2015 at 12:00 AM, Amit Kapila <amit.kapil...@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. With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com
