From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Euler Taveira <euler(at)eulerto(dot)com> |
Cc: | "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com> |
Subject: | Re: [BUG]Invalidate relcache when setting REPLICA IDENTITY |
Date: | 2021-11-13 10:49:41 |
Message-ID: | CAA4eK1JfAuXps=NmE02-wzc0wY+dQ8Ftz_gD0wjySNbmoePPeg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Nov 13, 2021 at 12:47 AM Euler Taveira <euler(at)eulerto(dot)com> wrote:
>
> On Fri, Nov 12, 2021, at 3:10 AM, houzj(dot)fnst(at)fujitsu(dot)com wrote:
>
> On Fri, Nov 12, 2021 1:33 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > On Fri, Nov 12, 2021 at 10:50 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > > But won't that generate invalidation for the rel twice in the case
> > > (change Replica Identity from Nothing to some index) you mentioned in
> > > the previous email?
> > >
> >
> > Oh, I see the point. I think this is okay because
> > AddRelcacheInvalidationMessage doesn't allow to add duplicate rel
> > invalidation. If that is the case I wonder why not simply register
> > invalidation without any check in the for loop as was the case with
> > Tang's original patch?
>
> OK, I also think the code in Tang's original patch is fine.
> Attach the patch which register invalidation without any check in the for loop.
>
> WFM.
>
The patch looks mostly good to me. I have slightly tweaked the
comments in the code (as per my previous suggestion) and test. Also, I
have slightly modified the commit message. If the attached looks good
to you then kindly prepare patches for back-branches.
--
With Regards,
Amit Kapila.
Attachment | Content-Type | Size |
---|---|---|
v7-0001-Invalidate-relcache-when-changing-REPLICA-IDENTIT.patch | application/octet-stream | 5.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bharath Rupireddy | 2021-11-13 12:57:02 | Re: Identify missing publications from publisher while create/alter subscription. |
Previous Message | Deng Kaisheng | 2021-11-13 08:41:34 | Anything I can contribute? |