Re: Logical replication can lose an update after concurrent index invalidation

From: vignesh C <vignesh21(at)gmail(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: Mihail Nikalayeu <mihailnikalayeu(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, onderkalaci(at)gmail(dot)com
Subject: Re: Logical replication can lose an update after concurrent index invalidation
Date: 2026-09-23 09:56:02
Message-ID: CALDaNm3m44D8O68iJoigYVLV0A5qmxBCbbABpd9bqDmkxVcwZA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, 22 Sept 2026 at 15:33, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Mon, Sep 21, 2026 at 6:01 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> >
> > Here is an updated version including the back-branches version.
> > This version also addresses Hou-san's agreed 1st comment from [1].
> >
>
> We should include the test till the version where required Injection
> Point functionality is present.

Here is an updated version with the injection point and its test added
up to the PG17 version, where injection points were introduced.
I have also retained the DROP INDEX CONCURRENTLY test instead of the
REINDEX CONCURRENTLY test, as the former requires only one injection
point (apply-update-before-open-indices), whereas the latter requires
two injection points: reindex-relation-concurrently-before-swap and
apply-update-before-open-indices.

Regards,
Vignesh

Attachment Content-Type Size
v6_REL17-0001-Fix-tuple-search-during-apply-after-concurr.patch application/octet-stream 22.6 KB
v6_REL18-0001-Fix-tuple-search-during-apply-after-concurr.patch application/octet-stream 23.2 KB
v6_REL16-0001-Fix-tuple-search-during-apply-after-concurr.patch application/octet-stream 15.2 KB
v6-0001-Fix-tuple-search-during-apply-after-concurrent-in.patch application/octet-stream 25.0 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Antonin Houska 2026-09-23 10:06:55 Re: REPACK enhancements
Previous Message Jeevan Chalke 2026-09-23 09:54:18 Re: Use-after-free of a shared Path in add_path()/add_partial_path()