On Thu, Oct 16, 2025 at 9:21 AM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>
> > Please check the attached v18.
>
> Thank you for updating the patch!
>
> I've reviewed the patch and here is one review comment:
>
> from->inh = false; /* apply ONLY */
> + if (get_rel_relkind(relid) == RELKIND_PARTITIONED_TABLE)
> + from->inh = true;
>
> It's better to check rel->rd_rel->relkind instead of calling
> get_rel_relkind() as it checks syscache.
>
> I've attached a patch to fix the above and includes some cosmetic
> changes. Please review it.
>
hi.
overall looks good to me, thanks for polishing it.
+ * However, when copying data from a partitioned table, we don't
+ * not use "ONLY", since we need to retrieve rows from its
+ * descendant tables too.
I guess here it should be
"we don't use "ONLY"
?
I’ve incorporated your changes into v19.