Re: [HACKERS] path toward faster partition pruning

On 2018/03/20 21:41, David Rowley wrote:
> On 21 March 2018 at 00:07, Amit Langote <Langote_Amit_f8@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