Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error
Date
Msg-id 20170829101302.g6q7exudl5btlzce@alvherre.pgsql
Whole thread Raw
In response to [HACKERS] [PATCH] Fix drop replication slot blocking instead of returning error  (Simone Gotti <simone.gotti@gmail.com>)
Responses Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error  (Simone Gotti <simone.gotti@gmail.com>)
List pgsql-hackers
Simone Gotti wrote:
> Hi all,
> 
> I noticed that in postgres 10beta3, calling pg_drop_replication_slot on an
> active slot will block until it's released instead of returning an error
> like
> done in pg 9.6. Since this is a change in the previous behavior and the docs
> wasn't changed I made a patch to restore the previous behavior.

Changing that behavior was the entire point of the cited commit.

A better fix, from my perspective, is to amend the docs as per the
attached patch.  This is what would be useful for logical replication,
which is what replication slots were invented for in the first place.
If you disagree, let's discuss what other use cases you have, and we can
come up with alternatives that satisfy both.  I think a decent answer,
but one which would create a bit of extra churn, would be to have an
optional boolean flag in the command/function for "nowait", instead of
hardcoding either behavior.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: [HACKERS] Re: [PATCH] Fix drop replication slot blocking insteadof returning error
Next
From: Ashutosh Bapat
Date:
Subject: Re: [HACKERS] Adding support for Default partition in partitioning