On Thursday, September 16, 2021 6:05 PM Amit Kapila <amit.kapil...@gmail.com> > > On Tuesday, September 14, 2021 10:41 PM vignesh C <vignes...@gmail.com> > > wrote: > > > On Tue, Sep 7, 2021 at 11:38 AM houzj.f...@fujitsu.com > > > <houzj.f...@fujitsu.com> wrote: > > > > Thanks for the comment. > > Attach new version patches which clean the table at the end. > > > > + * For partitioned table contained in the publication, we must > + * invalidate all partitions contained in the respective partition > + * trees, not just the one explicitly mentioned in the publication. > > Can we slightly change the above comment as: "For the partitioned tables, we > must invalidate all partitions contained in the respective partition > hierarchies, > not just the one explicitly mentioned in the publication. This is required > because we implicitly publish the child tables when the parent table is > published." > > Apart from this, the patch looks good to me. > > I think we need to back-patch this till v13. What do you think?
I agreed. Attach patches for back-branch, each has passed regression tests and pgindent. Best regards, Hou zj
HEAD-Invalidate-all-partitions-for-partitioned-table-in-p.patch
Description: HEAD-Invalidate-all-partitions-for-partitioned-table-in-p.patch
PG14-Invalidate-all-partitions-for-partitioned-table-in-p.patch
Description: PG14-Invalidate-all-partitions-for-partitioned-table-in-p.patch
PG13-Invalidate-all-partitions-for-partitioned-table-in-p.patch
Description: PG13-Invalidate-all-partitions-for-partitioned-table-in-p.patch
