On Wed, Sep 10, 2025 at 3:33 PM Rahila Syed <rahilasye...@gmail.com> 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.
>
I suggest removing validate_sync_standby_slots() entirely since there
isn’t much left in it.
The below logic currently in validate_sync_standby_slots() to parse
the list of slots can be directly moved into
check_synchronized_standby_slots().
/* Verify syntax and parse string into a list of identifiers */
ok = SplitIdentifierString(rawname, ',', elemlist);
if (!ok)
GUC_check_errdetail("List syntax is invalid.");
--
With Regards,
Ashutosh Sharma.
