pgsql: Fix catcache invalidation of a list entry that's being built - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix catcache invalidation of a list entry that's being built
Date
Msg-id E1tXgjB-001Vbb-JS@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix catcache invalidation of a list entry that's being built

If a new catalog tuple is inserted that belongs to a catcache list
entry, and cache invalidation happens while the list entry is being
built, the list entry might miss the newly inserted tuple.

To fix, change the way we detect concurrent invalidations while a
catcache entry is being built. Keep a stack of entries that are being
built, and apply cache invalidation to those entries in addition to
the real catcache entries. This is similar to the in-progress list in
relcache.c.

Back-patch to all supported versions.

Reviewed-by: Noah Misch
Discussion: https://www.postgresql.org/message-id/2234dc98-06fe-42ed-b5db-ac17384dc880@iki.fi

Branch
------
REL_16_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/91fc447c21d38f6631480ae6f252dbf4211b10b2

Modified Files
--------------
src/backend/utils/cache/catcache.c | 231 +++++++++++++++++++++++--------------
src/backend/utils/cache/inval.c    |   2 +-
src/include/utils/catcache.h       |   1 +
src/tools/pgindent/typedefs.list   |   1 +
4 files changed, 148 insertions(+), 87 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Bump PGSTAT_FILE_FORMAT_ID
Next
From: Dean Rasheed
Date:
Subject: pgsql: psql: Add leakproof indicator to \df+, \do+, \dAo+, and \dC+ out