pgsql: Fix updates of indisvalid for partitioned indexes - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Fix updates of indisvalid for partitioned indexes
Date
Msg-id E1qK7Nr-000FaG-8L@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix updates of indisvalid for partitioned indexes

indisvalid is switched to true for partitioned indexes when all its
partitions have valid indexes when attaching a new partition, up to the
top-most parent if all its leaves are themselves valid when dealing with
multiple layers of partitions.

The copy of the tuple from pg_index used to switch indisvalid to true
came from the relation cache, which is incorrect.  Particularly, in the
case reported by Shruthi Gowda, executing a series of commands in a
single transaction would cause the validation of partitioned indexes to
use an incorrect version of a pg_index tuple, as indexes are reloaded
after an invalidation request with RelationReloadIndexInfo(), a much
faster version than a full index cache rebuild.  In this case, the
limited information updated in the cache leads to an incorrect version
of the tuple used.  One of the symptoms reported was the following
error, with a replica identity update, for instance:
"ERROR: attempted to update invisible tuple"

This is incorrect since 8b08f7d, so backpatch all the way down.

Reported-by: Shruthi Gowda
Author: Michael Paquier
Reviewed-by: Shruthi Gowda, Dilip Kumar
Discussion: https://postgr.es/m/CAASxf_PBcxax0wW-3gErUyftZ0XrCs3Lrpuhq4-Z3Fak1DoW7Q@mail.gmail.com
Backpatch-through: 11

Branch
------
REL_16_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/31f9d41d625dc6ca8c1469edd6919316b18174fe

Modified Files
--------------
src/backend/commands/tablecmds.c       | 15 +++++---
src/test/regress/expected/indexing.out | 63 ++++++++++++++++++++++++++++++++++
src/test/regress/sql/indexing.sql      | 43 +++++++++++++++++++++++
3 files changed, 117 insertions(+), 4 deletions(-)


pgsql-committers by date:

Previous
From: Thomas Munro
Date:
Subject: pgsql: Remove wal_sync_method=fsync_writethrough on Windows.
Next
From: Michael Paquier
Date:
Subject: pgsql: Add indisreplident to fields refreshed by RelationReloadIndexInf