]> granicus.if.org Git - postgresql/commit
Fix a bug in how we generate partition constraints.
authorRobert Haas <rhaas@postgresql.org>
Fri, 13 Jan 2017 19:03:52 +0000 (14:03 -0500)
committerRobert Haas <rhaas@postgresql.org>
Fri, 13 Jan 2017 19:04:35 +0000 (14:04 -0500)
commit0563a3a8b59150bf3cc8b2b7077f684e0eaf8aff
treeb28efdc0bad75090561d1075f76d7bf1720f1f8f
parent0c2070cefa0e5d097b715c9a3b9b5499470019aa
Fix a bug in how we generate partition constraints.

Move the code for doing parent attnos to child attnos mapping for Vars
in partition constraint expressions to a separate function
map_partition_varattnos() and call it from the appropriate places.
Doing it in get_qual_from_partbound(), as is now, would produce wrong
result in certain multi-level partitioning cases, because it only
considers the current pair of parent-child relations.  In certain
multi-level partitioning cases, attnums for the same key attribute(s)
might differ between various levels causing the same attribute to be
numbered differently in different instances of the Var corresponding
to a given attribute.

With this commit, in generate_partition_qual(), we first generate the
the whole partition constraint (considering all levels of partitioning)
and then do the mapping, so that Vars in the final expression are
numbered according the leaf relation (to which it is supposed to apply).

Amit Langote, reviewed by me.
src/backend/catalog/partition.c
src/backend/commands/tablecmds.c
src/include/catalog/partition.h
src/test/regress/expected/alter_table.out
src/test/regress/sql/alter_table.sql