pgsql: Fix dependency handling at swap phase of REINDEX CONCURRENTLY - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Fix dependency handling at swap phase of REINDEX CONCURRENTLY
Date
Msg-id E1iOvTO-00053L-PT@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix dependency handling at swap phase of REINDEX CONCURRENTLY

When swapping the dependencies of the old and new indexes, the code has
been correctly switching all links in pg_depend from the old to the new
index for both referencing and referenced entries.  However it forgot
the fact that the new index may itself have existing entries in
pg_depend, like references to the parent table attributes.  This
resulted in duplicated entries in pg_depend after running REINDEX
CONCURRENTLY.

Fix this problem by removing any existing entries in pg_depend on the
new index before switching the dependencies of the old index to the new
one.  More regression tests are added to check the consistency of
entries in pg_depend for indexes, including partition indexes.

Author: Michael Paquier
Discussion: https://postgr.es/m/20191025064318.GF8671@paquier.xyz
Backpatch-through: 12

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/68ac9cf2499236996f3d4bf31f7f16d5bd3c77af

Modified Files
--------------
src/backend/catalog/index.c                |   6 +-
src/test/regress/expected/create_index.out | 126 +++++++++++++++++++++++++++++
src/test/regress/sql/create_index.sql      |  54 +++++++++++++
3 files changed, 185 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Fix initialization of fake LSN for unlogged relations
Next
From: Michael Paquier
Date:
Subject: pgsql: Doc: Add missing step for pg_stat_progress_cluster