Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error
Дата
Msg-id 20170829101302.g6q7exudl5btlzce@alvherre.pgsql
обсуждение исходный текст
Ответ на [HACKERS] [PATCH] Fix drop replication slot blocking instead of returning error  (Simone Gotti <simone.gotti@gmail.com>)
Ответы Re: [HACKERS] [PATCH] Fix drop replication slot blocking instead ofreturning error  (Simone Gotti <simone.gotti@gmail.com>)
Список 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

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: [HACKERS] Re: [PATCH] Fix drop replication slot blocking insteadof returning error
Следующее
От: Ashutosh Bapat
Дата:
Сообщение: Re: [HACKERS] Adding support for Default partition in partitioning