]> granicus.if.org Git - postgresql/commit
Fix handling of expressions and predicates in REINDEX CONCURRENTLY
authorMichael Paquier <michael@paquier.xyz>
Mon, 29 Jul 2019 00:58:49 +0000 (09:58 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 29 Jul 2019 00:58:49 +0000 (09:58 +0900)
commit7cce159349ccdb39ade07f869f08e4929ef2fe0b
tree6921d96eebc4a09bb5df3426874937720dd5e86a
parenta2a777d011971ace3a349a3f02b1bf6eeea07bf2
Fix handling of expressions and predicates in REINDEX CONCURRENTLY

When copying the definition of an index rebuilt concurrently for the new
entry, the index information was taken directly from the old index using
the relation cache.  In this case, predicates and expressions have
some post-processing to prepare things for the planner, which loses some
information including the collations added in any of them.

This inconsistency can cause issues when attempting for example a table
rewrite, and makes the new indexes rebuilt concurrently inconsistent
with the old entries.

In order to fix the problem, fetch expressions and predicates directly
from the catalog of the old entry, and fill in IndexInfo for the new
index with that.  This makes the process more consistent with
DefineIndex(), and the code is refactored with the addition of a routine
to create an IndexInfo node.

Reported-by: Manuel Rigger
Author: Michael Paquier
Discussion: https://postgr.es/m/CA+u7OA5Hp0ra235F3czPom_FyAd-3+XwSJmX95r1+sRPOJc9VQ@mail.gmail.com
Backpatch-through: 12
src/backend/catalog/index.c
src/backend/commands/indexcmds.c
src/backend/nodes/makefuncs.c
src/include/nodes/makefuncs.h
src/test/regress/expected/create_index.out
src/test/regress/sql/create_index.sql