Horton

Re: Selectively invalidate caches in pgoutput module

Amit Kapila <amit.kapila16@gmail.com>

View thread View on postgr.es
From: Amit Kapila <amit.kapila16@gmail.com>
To: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>
Cc: "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>, "ShlokKumar.Kyal@fujitsu.com" <ShlokKumar.Kyal@fujitsu.com>, "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>, Masahiko Sawada <sawada.mshk@gmail.com>
Date: 2025-03-13T08:43:39Z
Lists: pgsql-hackers 
On Wed, Mar 12, 2025 at 5:46 PM Hayato Kuroda (Fujitsu)
<kuroda.hayato@fujitsu.com> wrote:
>
> > Thanks, the patch looks mostly good to me. I have made few cosmetic
> > changes in the attached and combined both the patches.
>
> Thanks, it looks good to me.
>
> > The existing
> > Alter Publication ... Rename tests don't test invalidations arising
> > from that command. As this patch changes that code path, it would be
> > good to add a few tests for the same. We can add one for individual
> > relations and another for ALL Tables publication.
>
> I created new patch which adds a test code.
>

Thanks. I have pushed the patch after minor changes in the test.

-- 
With Regards,
Amit Kapila.

Commits

  1. Avoid invalidating all RelationSyncCache entries on publication rename.

  2. Avoid invalidating all RelationSyncCache entries on publication change.