Re: Add an optional timeout clause to isolationtester step.

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Add an optional timeout clause to isolationtester step.
Дата
Msg-id 20200310021459.GA4369@paquier.xyz
обсуждение исходный текст
Ответ на Re: Add an optional timeout clause to isolationtester step.  (Andres Freund <andres@anarazel.de>)
Ответы Re: Add an optional timeout clause to isolationtester step.  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Mon, Mar 09, 2020 at 03:15:58PM -0700, Andres Freund wrote:
> On 2020-03-07 22:17:09 +0100, Julien Rouhaud wrote:
>> For reindex concurrently, a SELECT FOR UPDATE on a different connection can
>> ensure that the reindex will be stuck at some point, so canceling the command
>> after a long enough timeout reproduces the original faulty behavior.
>
> That kind of thing can already be done using statement_timeout or
> lock_timeout, no?

Yep, still that's not something I would recommend to commit in the
tree as that's a double-edged sword as you already know.  For slower
machines, you need a statement_timeout large enough so as you make
sure that the state you want the query to wait for is reached, which
has a cost on all other faster machines as it makes the tests slower.
--
Michael

Вложения

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

Предыдущее
От: James Coleman
Дата:
Сообщение: Re: Nicer error when connecting to standby with hot_standby=off
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: Proposal: Add more compile-time asserts to exposeinconsistencies.