Re: Unexpected Standby Shutdown on sync_replication_slots change
Laurenz Albe <laurenz.albe@cybertec.at>
From: Laurenz Albe <laurenz.albe@cybertec.at>
To: Fujii Masao <masao.fujii@gmail.com>, Amit Kapila
<amit.kapila16@gmail.com>
Cc: shveta malik <shveta.malik@gmail.com>, Hugo DUBOIS
<hdubois@scaleway.com>, pgsql-bugs@lists.postgresql.org
Date: 2025-07-29T06:34:52Z
Lists: pgsql-bugs
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Disallow server start with sync_replication_slots = on and wal_level < logical.
- 12da45742cfd 19 (unreleased) landed
-
Avoid unexpected shutdown when sync_replication_slots is enabled.
- f71fa981c96d 17.6 landed
- 2d81a246f415 18.0 landed
- 4614d53d4ef4 19 (unreleased) landed
On Tue, 2025-07-29 at 13:40 +0900, Fujii Masao wrote: > I think it's basically not acceptable for a server to start up > successfully before a minor version update, but then fail to start with > the same configuration after the update. If that's absolutely necessary > to fix a bug, it might be justifiable. But in this case, I don't think > it's required. > > Blocking startup when sync_replication_slots is enabled and wal_level > is not logical could be helpful. But that feels more like an improvement > than a bug fix. I'm fine adding that to master, but I don't think we should > apply it to old branches. > > That said, both Shveta and Amit support backpatching this change, > so I'd like to hear more opinions before we decide. I side with you on that one. I think it would be fine to backpatch the breaking change to v18, if we mention the behavior change in the release notes. Yours, Laurenz Albe