Close race condition between datfrozen and relfrozen updates.
authorNoah Misch <noah@leadboat.com>
Mon, 29 Apr 2024 17:24:56 +0000 (10:24 -0700)
committerNoah Misch <noah@leadboat.com>
Mon, 29 Apr 2024 17:25:00 +0000 (10:25 -0700)
commit70cadfba0c70b25922d0739a9e09d24f0efd7f46
tree245778aa706a272c45bc41875b3628dfed7702bb
parent440b6251b75220bd28c063f50f5031f2004758f6
Close race condition between datfrozen and relfrozen updates.

vac_update_datfrozenxid() did multiple loads of relfrozenxid and
relminmxid from buffer memory, and it assumed each would get the same
value.  Not so if a concurrent vac_update_relstats() did an inplace
update.  Commit 2d2e40e3befd8b9e0d2757554537345b15fa6ea2 fixed the same
kind of bug in vac_truncate_clog().  Today's bug could cause the
rel-level field and XIDs in the rel's rows to precede the db-level
field.  A cluster having such values should VACUUM affected tables.
Back-patch to v12 (all supported versions).

Discussion: https://postgr.es/m/20240423003956.e7.nmisch@google.com
src/backend/commands/vacuum.c