]> git.ipfire.org Git - thirdparty/postgresql.git/commit
At end of recovery, reset all sinval-managed caches.
authorNoah Misch <noah@leadboat.com>
Fri, 25 Oct 2024 13:51:06 +0000 (06:51 -0700)
committerNoah Misch <noah@leadboat.com>
Fri, 25 Oct 2024 13:51:06 +0000 (06:51 -0700)
commitd36b4d8ec3227c1406c2a3e046153b32b33cecd8
tree45526e73a7fc0d776e4af689bbbb79af6cadd021
parent2d63c964f0104337ba109a96304491df25e167f2
At end of recovery, reset all sinval-managed caches.

An inplace update's invalidation messages are part of its transaction's
commit record.  However, the update survives even if its transaction
aborts or we stop recovery before replaying its transaction commit.
After recovery, a backend that started in recovery could update the row
without incorporating the inplace update.  That could result in a table
with an index, yet relhasindex=f.  That is a source of index corruption.

This bulk invalidation avoids the functional consequences.  A future
change can fix the !RecoveryInProgress() scenario without changing the
WAL format.  Back-patch to v17 - v12 (all supported versions).  v18 will
instead add invalidations to WAL.

Discussion: https://postgr.es/m/20240618152349.7f.nmisch@google.com
src/backend/access/transam/xlog.c
src/backend/storage/ipc/sinvaladt.c
src/include/storage/sinvaladt.h