| 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...
| 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. |
| 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 |