On 2018/03/20 21:41, David Rowley wrote: > On 21 March 2018 at 00:07, Amit Langote <langote_amit...@lab.ntt.co.jp> wrote: >> Attached is further revised version. > > In the 0004 patch I see: > > @@ -1439,6 +1441,10 @@ inheritance_planner(PlannerInfo *root) > if (IS_DUMMY_PATH(subpath)) > continue; > > + /* Add the current parent's RT index to the partitioned rels set. */ > + partitioned_relids = bms_add_member(partitioned_relids, > + appinfo->parent_relid); > > This seems to execute regardless of if the target relation is a > partitioned table or an inheritance parent. I think there needs to be > a condition so you only do this when planning for partitioned tables.
Oops, that's quite wrong. Will fix, thanks. Regards, Amit
