Re: pgsql: Prevent invalidation of newly synced replication slots.

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Amit Kapila <akapila(at)postgresql(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Prevent invalidation of newly synced replication slots.
Date: 2026-01-27 16:16:13
Message-ID: CA+hUKGJotCdMdhrwVST4xbsPzr-csDgzveNhKu4UAXEmCDJ4iA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Wed, Jan 28, 2026 at 3:59 AM Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> I imagine this is going to break CI for everybody else too, as well as cfbot.

Just by the way, on that last point, we trained cfbot to watch out for
CI pass/fail in this account:

https://github.com/postgres/postgres/commits/master/

and then use the most recent pass as the base commit when applying
patches to make test branches. So if master is broken for a while, it
no longer takes all the cfbot runs with it. Mentioning just in case
anyone is confused by that...

As for what's happening... hmm, there are a few holes in the "shared
locking" stuff you get with the flags we use. For example you can't
unlink a directory that contains a file that has been unlinked but
someone still holds open. Doesn't seem to be the case here. But I
wonder if you can't rename("old", "new") where "new" is a file that
has already been unlinked (or renamed over) that someone still holds
open, or something like that...

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Jeff Davis 2026-01-27 16:16:26 pgsql: Fix crash introduced by incorrect backport 806555e300.
Previous Message Tom Lane 2026-01-27 16:11:08 Re: pgsql: Prevent invalidation of newly synced replication slots.

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2026-01-27 16:17:31 Re: pgsql: Prevent invalidation of newly synced replication slots.
Previous Message Peter Eisentraut 2026-01-27 16:15:42 Re: RepOrigin vs. replorigin