Remaining references to RecentGlobalXmin

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Remaining references to RecentGlobalXmin
Дата
Msg-id YKtMFDjZfR+gySnh@paquier.xyz
обсуждение исходный текст
Ответы Re: Remaining references to RecentGlobalXmin  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
Hi,

dc7420c2 has removed RecentGlobalXmin, but there are still references
to it in the code, and a set of FIXME references, like this one in
autovacuum.c (three in total):
/*
 * Start a transaction so we can access pg_database, and get a snapshot.
 * We don't have a use for the snapshot itself, but we're interested in
 * the secondary effect that it sets RecentGlobalXmin.  (This is critical
 * for anything that reads heap pages, because HOT may decide to prune
 * them even if the process doesn't attempt to modify any tuples.)
 *
 * FIXME: This comment is inaccurate / the code buggy. A snapshot that is
 * not pushed/active does not reliably prevent HOT pruning (->xmin could
 * e.g. be cleared when cache invalidations are processed).
 */

Wouldn't it be better to clean up that?
Thanks,
--
Michael

Вложения

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

Предыдущее
От: "osumi.takamichi@fujitsu.com"
Дата:
Сообщение: RE: Forget close an open relation in ReorderBufferProcessTXN()
Следующее
От: Bharath Rupireddy
Дата:
Сообщение: Re: Refactor "mutually exclusive options" error reporting code in parse_subscription_options