RE: tablesync copy ignores publication actions - Mailing list pgsql-hackers

From shiy.fnst@fujitsu.com
Subject RE: tablesync copy ignores publication actions
Date
Msg-id OSZPR01MB63104AD8B1AD1E72FE1969B6FDB59@OSZPR01MB6310.jpnprd01.prod.outlook.com
Whole thread Raw
In response to Re: tablesync copy ignores publication actions  (Peter Smith <smithpb2250@gmail.com>)
Responses Re: tablesync copy ignores publication actions
List pgsql-hackers
On Wed, Jun 22, 2022 4:49 PM Peter Smith <smithpb2250@gmail.com> wrote:
> 
> On Wed, Jun 22, 2022 at 2:18 PM Amit Kapila <amit.kapila16@gmail.com>
> wrote:
> >
> > On Thu, Jun 16, 2022 at 6:07 AM Peter Smith <smithpb2250@gmail.com>
> wrote:
> >
> > >
> > > Thank you for your review comments. Those reported mistakes are fixed
> > > in the attached patch v3.
> > >
> >
> > This patch looks mostly good to me except for a few minor comments
> > which are mentioned below. It is not very clear in which branch(es) we
> > should commit this patch? As per my understanding, this is a
> > pre-existing behavior but we want to document it because (a) It was
> > not already documented, and (b) we followed it for row filters in
> > PG-15 it seems that should be explained. So, we have the following
> > options (a) commit it only for PG-15, (b) commit for PG-15 and
> > backpatch the relevant sections, or (c) commit it when branch opens
> > for PG-16. What do you or others think?
> 
> Even though this is a very old docs omission, AFAIK nobody ever raised
> it as a problem before. It only became more important because of the
> PG15 row-filters. So I think option (a) is ok.
> 

I also think option (a) is ok.

> 
> PSA patch v4 to address all the above review comments.
> 

Thanks for updating the patch. It looks good to me.

Besides, I tested the examples in the patch, and there's no problem.

Regards,
Shi yu

pgsql-hackers by date:

Previous
From: Julien Rouhaud
Date:
Subject: Re: [PROPOSAL] Detecting plan changes with plan_id in pg_stat_activity
Next
From: Michael Paquier
Date:
Subject: Re: Add header support to text format and matching feature