pgsql: Change the way ATExecMergePartitions() handles the name collisio - Mailing list pgsql-committers

From Alexander Korotkov
Subject pgsql: Change the way ATExecMergePartitions() handles the name collisio
Date
Msg-id E1s1jM9-000lFj-Hl@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Change the way ATExecMergePartitions() handles the name collision

The name collision happens when the name of the new partition is the same as
the name of one of the merging partitions.  Currently, ATExecMergePartitions()
first gives the new partition a temporary name and then renames it when old
partitions are deleted.  That negatively influences the naming of related
objects like indexes and constrains, which could inherit a temporary name.

This commit changes the implementation in the following way.  A merging
partition gets renamed first, then the new partition is created with the
right name immediately.  This resolves the issue of the naming of related
objects.

Reported-by: Alexander Lakhin
Discussion: https://postgr.es/m/edfbd846-dcc1-42d1-ac26-715691b687d3%40postgrespro.ru
Author: Dmitry Koval, Alexander Korotkov
Reviewed-by: Robert Haas, Justin Pryzby, Pavel Borisov

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/885742b9f88b9386368ee94df8c94d154677ffba

Modified Files
--------------
src/backend/commands/tablecmds.c              | 63 +++++++++++++--------------
src/test/regress/expected/partition_merge.out | 25 +++++++++++
src/test/regress/sql/partition_merge.sql      | 18 ++++++++
3 files changed, 73 insertions(+), 33 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgsql: Reject SSL connection if ALPN is used but there's no common prot
Next
From: Alexander Korotkov
Date:
Subject: pgsql: Stabilize regression tests introduced by 259c96fa8f