From 1e09473bb2f571af0a6a8e4ca718d291efa63772 Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Tue, 30 May 2017 14:57:01 -0400 Subject: [PATCH] Remove replication slot name check from ReplicationSlotAcquire() When trying to access a replication slot that is supposed to already exist, we don't need to check the naming rules again. If the slot does not exist, we will then get a "does not exist" error message, which is generally more useful from the perspective of an end user. --- src/backend/replication/slot.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/backend/replication/slot.c b/src/backend/replication/slot.c index 5386e86aa6..c0f7fbb2b2 100644 --- a/src/backend/replication/slot.c +++ b/src/backend/replication/slot.c @@ -331,8 +331,6 @@ ReplicationSlotAcquire(const char *name) Assert(MyReplicationSlot == NULL); - ReplicationSlotValidateName(name, ERROR); - /* Search for the named slot and mark it active if we find it. */ LWLockAcquire(ReplicationSlotControlLock, LW_SHARED); for (i = 0; i < max_replication_slots; i++) -- 2.13.0