Re: Autovacuum and idle_session_timeout

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: Autovacuum and idle_session_timeout
Дата
Msg-id CAECtzeXKgf=N4DLRkze9Zk6L=x3kCVXoNEF1Qqs2KTAnk7UZsw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Autovacuum and idle_session_timeout  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Le jeu. 30 déc. 2021 à 17:25, Tom Lane <tgl@sss.pgh.pa.us> a écrit :
Japin Li <japinli@hotmail.com> writes:
> On Thu, 30 Dec 2021 at 18:53, Guillaume Lelarge <guillaume@lelarge.info> wrote:
>> pg_dump works in a single transaction, so it's already dealt with
>> idle_in_transaction_timeout. Though I guess setting both would work too.

> Attached fix this, please consider reveiew it.  Thanks.

This seems rather pointless to me.  The idle-session timeout is only
activated in PostgresMain's input loop, so it will never be reached
in autovacuum or other background workers.  (The same is true for
idle_in_transaction_session_timeout, so the fact that somebody made
autovacuum.c clear that looks like cargo-cult programming from here,
not useful code.)  And as for pg_dump, how would it ever trigger the
timeout?  It's not going to sit there thinking, especially not
outside a transaction.


Agreed. It makes more sense. So no need for the patch. Thanks to both.


--
Guillaume.

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

Предыдущее
От: "Bossart, Nathan"
Дата:
Сообщение: Re: Strange path from pgarch_readyXlog()
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Throttling WAL inserts when the standby falls behind more than the configured replica_lag_in_bytes