pgsql: Don't allow to set replication slot_name as ''.

From: Amit Kapila <akapila(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Don't allow to set replication slot_name as ''.
Date: 2021-07-19 06:10:21
Message-ID: E1m5MTt-0005nk-3n@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Don't allow to set replication slot_name as ''.

We don't allow to create replication slot_name as an empty string ('') via
SQL API pg_create_logical_replication_slot() but it is allowed to be set
via Alter Subscription command. This will lead to apply worker repeatedly
keep trying to stream data via slot_name '' and the user is not allowed to
create the slot with that name.

Author: Japin Li
Reviewed-By: Ranier Vilela, Amit Kapila
Backpatch-through: 10, where it was introduced
Discussion: https://postgr.es/m/MEYP282MB1669CBD98E721C77CA696499B61A9@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/29abde637bbb4b8fcdc96e049813890afa5ab671

Modified Files
--------------
src/backend/commands/subscriptioncmds.c | 2 ++
src/test/regress/expected/subscription.out | 3 +++
src/test/regress/sql/subscription.sql | 3 +++
3 files changed, 8 insertions(+)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2021-07-19 06:25:25 pgsql: Use l*_node() family of functions where appropriate
Previous Message Thomas Munro 2021-07-19 04:52:08 pgsql: Adjust commit 2dbe8905 for ancient macOS.