Re: RecoveryInProgress() has critical side effects

Поиск
Список
Период
Сортировка
От Bossart, Nathan
Тема Re: RecoveryInProgress() has critical side effects
Дата
Msg-id FE730879-BCD5-44E8-9FD6-7DAA5E06C948@amazon.com
обсуждение исходный текст
Ответ на Re: RecoveryInProgress() has critical side effects  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: RecoveryInProgress() has critical side effects  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
On 11/15/21, 1:30 PM, "Robert Haas" <robertmhaas@gmail.com> wrote:
> Here's a new version that does it that way. Any other opinions?

LGTM

> The best thing I could come up with for a test case for this was to
> try repeatedly making a new connection and running "SELECT
> txid_current()", which will cause just one WAL record to be generated.
> Unfortunately that path has overhead from a lot of other causes so I'm
> not sure the results are very meaningful, but here they are:
>
> v1: 0.378 ms
> v2: 0.391 ms
> common base commit (10eae82b2): 0.376 ms

I'm personally not too worried about a ~4% regression in this
particular benchmark...

Nathan


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

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: [RFC] building postgres with meson
Следующее
От: "Bossart, Nathan"
Дата:
Сообщение: Re: [PATCH v2] use has_privs_for_role for predefined roles