On Thursday, December 2, 2021 12:50 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > On Thu, Dec 2, 2021 at 9:41 AM Greg Nancarrow <gregn4...@gmail.com> > wrote: > > > > On Thu, Dec 2, 2021 at 1:48 PM Greg Nancarrow <gregn4...@gmail.com> > wrote: > > > > > > If you updated my original description to say "(instead of just the > > > individual partitions)", it would imply the same I think. > > > But I don't mind if you want to explicitly state both cases to make it > > > clear. > > > > > > > For example, something like: > > > > For publications of partitioned tables with publish_via_partition_root > > set to true, only the partitioned table (and not its partitions) is > > included in the view, whereas if publish_via_partition_root is set to > > false, only the individual partitions are included in the view. > > > > Yeah, that sounds good to me.
It looks good to me as well. Attach the patches for (HEAD~13) which merge the suggested doc change. I prepared the code patch and test patch separately to make it easier for committer to confirm. Best regards, Hou zj
HEAD-0001-Fix-double-publish-of-child-table-s-data.patch
Description: HEAD-0001-Fix-double-publish-of-child-table-s-data.patch
PG14-0001-Fix-double-publish-of-child-table-s-data.patch
Description: PG14-0001-Fix-double-publish-of-child-table-s-data.patch
PG13-0001-Fix-double-publish-of-child-table-s-data.patch
Description: PG13-0001-Fix-double-publish-of-child-table-s-data.patch
HEAD-0002-testcases.patch
Description: HEAD-0002-testcases.patch
PG13-0002-testcases.patch
Description: PG13-0002-testcases.patch
PG14-0002-testcases.patch
Description: PG14-0002-testcases.patch
