diff options
author | Michael Paquier <michael@paquier.xyz> | 2023-07-14 11:16:11 +0900 |
---|---|---|
committer | Michael Paquier <michael@paquier.xyz> | 2023-07-14 11:16:11 +0900 |
commit | 7d27493b74c5893fb50b82266d019964493318d9 (patch) | |
tree | 4b32a512a2a730ecd702b4258344fcacffc33836 /src/backend/utils/cache | |
parent | f1d6bcdd8fb5db1f71ea0e55ade2221c3c72c96e (diff) | |
download | postgresql-7d27493b74c5893fb50b82266d019964493318d9.tar.gz postgresql-7d27493b74c5893fb50b82266d019964493318d9.zip |
Add indisreplident to fields refreshed by RelationReloadIndexInfo()
RelationReloadIndexInfo() is a fast-path used for index reloads in the
relation cache, and it has always forgotten about updating
indisreplident, which is something that would happen after an index is
selected for a replica identity. This can lead to incorrect cache
information provided when executing a command in a transaction context
that updates indisreplident.
None of the code paths currently on HEAD that need to check upon
pg_index.indisreplident fetch its value from the relation cache, always
relying on a fresh copy on the syscache. Unfortunately, this may not be
the case of out-of-core code, that could see out-of-date value.
Author: Shruthi Gowda
Reviewed-by: Robert Haas, Dilip Kumar, Michael Paquier
Discussion: https://postgr.es/m/CAASxf_PBcxax0wW-3gErUyftZ0XrCs3Lrpuhq4-Z3Fak1DoW7Q@mail.gmail.com
Backpatch-through: 11
Diffstat (limited to 'src/backend/utils/cache')
-rw-r--r-- | src/backend/utils/cache/relcache.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c index 4b747550ca0..0f9f3b3d130 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -2288,6 +2288,7 @@ RelationReloadIndexInfo(Relation relation) relation->rd_index->indcheckxmin = index->indcheckxmin; relation->rd_index->indisready = index->indisready; relation->rd_index->indislive = index->indislive; + relation->rd_index->indisreplident = index->indisreplident; /* Copy xmin too, as that is needed to make sense of indcheckxmin */ HeapTupleHeaderSetXmin(relation->rd_indextuple->t_data, |