Re: [PATCH] Fix drop replication slot blocking instead of returning error

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Simone Gotti <simone(dot)gotti(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCH] Fix drop replication slot blocking instead of returning error
Date: 2017-08-29 10:13:02
Message-ID: 20170829101302.g6q7exudl5btlzce@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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

Attachment Content-Type Size
0001-Document-that-DROP_REPLICATION_SLOT-now-waits.patch text/plain 1.0 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2017-08-29 11:01:27 Re: Adding support for Default partition in partitioning
Previous Message Alvaro Herrera 2017-08-29 10:04:33 Re: Re: [PATCH] Fix drop replication slot blocking instead of returning error