On Wed, 10 Sept 2025 at 15:33, Rahila Syed <[email protected]> wrote: > > Hi, > > >> As for the synchronized_standby_slots, we can follow the behavior >> similar to check_synchronous_standby_names and just give parsing >> ERRORs. Any non-existent slot related errors can be given when that >> parameter is later used. >> >> -- > > > Please find attached a patch that implements this. I will work on adding a > test for it. > Hi Rahila,
I think we should also add a parsing check for slot names specified in
the GUC synchronize_standby_slots as suggested by Amit in [1].
I think we should also update the comment message in function
StandbySlotsHaveCaughtup
/*
* If a slot name provided in synchronized_standby_slots does not
* exist, report a message and exit the loop.
*
* Though validate_sync_standby_slots (the GUC check_hook) tries to
* avoid this, it can nonetheless happen because the user can specify
* a nonexistent slot name before server startup. That function cannot
* validate such a slot during startup, as ReplicationSlotCtl is not
* initialized by then. Also, the user might have dropped one slot.
*/
I made the changes in the above for the same and attached the updated patch.
[1]:
https://www.postgresql.org/message-id/CAA4eK1Jprqfs_URBmZ5%3DOOL98D05rPiGup1sscWgcCGcWU%3D9iA%40mail.gmail.com
Thanks,
Shlok Kyal
v2-0001-Remove-the-validation-from-the-GUC-check-hook-and.patch
Description: Binary data
