RE: issue with synchronized_standby_slots
Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
From: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>
To: 'Shlok Kyal' <shlok.kyal.oss@gmail.com>
Cc: Fabrice Chapuis <fabrice636861@gmail.com>, Ashutosh Sharma <ashu.coek88@gmail.com>, Rahila Syed <rahilasyed90@gmail.com>, Alexander
Kukushkin <cyberdemn@gmail.com>, "Zhijie Hou (Fujitsu)" <houzj.fnst@fujitsu.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Amit Kapila <amit.kapila16@gmail.com>
Date: 2025-10-23T08:15:01Z
Lists: pgsql-hackers
Dear Shlok, Thanks for updating the patch! Few comments. 1. You must separate patch for master and PG18, because ReplicationSlotValidateNameInternal() does not accept `bool allow_reserved_name` in the version. 2. Also, test for PG18 should not have the case which rejects the reserved name. 3. ``` -- Parallel worker does not throw error during startup. SET min_parallel_table_scan_size TO 0; SET parallel_setup_cost TO 0; SET parallel_tuple_cost TO 0; ``` According to contrib/pg_stat_statements/sql/parallel.sql, max_parallel_workers_per_gather should be also set. There is a possiblity that `make installcheck` is used and it has max_parallel_workers_per_gather=0. 4. ``` foreach_ptr(char, name, *elemlist) ``` You can add a comment atop here like: Iterate the list to validate each slot name. Best regards, Hayato Kuroda FUJITSU LIMITED