]> granicus.if.org Git - postgresql/commitdiff
Don't mark partitioned indexes invalid unnecessarily
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 5 Dec 2018 16:31:51 +0000 (13:31 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Wed, 5 Dec 2018 16:31:51 +0000 (13:31 -0300)
When an indexes is created on a partitioned table using ONLY (don't
recurse to partitions), it gets marked invalid until index partitions
are attached for each table partition.  But there's no reason to do this
if there are no partitions ... and moreover, there's no way to get the
index to become valid afterwards, because all partitions that get
created/attached get their own index partition already attached to the
parent index, so there's no chance to do ALTER INDEX ... ATTACH PARTITION
that would make the parent index valid.

Fix by not marking the index as invalid to begin with.

This is very similar to 9139aa19423b, but the pg_dump aspect does not
appear to be relevant until we add FKs that can point to PKs on
partitioned tables.  (I tried to cause the pg_upgrade test to break by
leaving some of these bogus tables around, but wasn't able to.)

Making this change means that an index that was supposed to be invalid
in the insert_conflict regression test is no longer invalid; reorder the
DDL so that the test continues to verify the behavior we want it to.

Author: Álvaro Herrera
Reviewed-by: Amit Langote
Discussion: https://postgr.es/m/20181203225019.2vvdef2ybnkxt364@alvherre.pgsql

src/backend/commands/indexcmds.c
src/test/regress/expected/insert_conflict.out
src/test/regress/sql/insert_conflict.sql

index 73656d8cc8466bf60272904ca76f1b7f83121e13..6c06167fb2ae39e81e0cf64e6069cceb9e689dca 100644 (file)
@@ -833,8 +833,18 @@ DefineIndex(Oid relationId,
                flags |= INDEX_CREATE_PARTITIONED;
        if (stmt->primary)
                flags |= INDEX_CREATE_IS_PRIMARY;
+
+       /*
+        * If the table is partitioned, and recursion was declined but partitions
+        * exist, mark the index as invalid.
+        */
        if (partitioned && stmt->relation && !stmt->relation->inh)
-               flags |= INDEX_CREATE_INVALID;
+       {
+               PartitionDesc   pd = RelationGetPartitionDesc(rel);
+
+               if (pd->nparts != 0)
+                       flags |= INDEX_CREATE_INVALID;
+       }
 
        if (stmt->deferrable)
                constr_flags |= INDEX_CONSTR_CREATE_DEFERRABLE;
index 12ae6c5fa5f4f8d78a4a1e0f3ec6ebf52d727e10..1338b2b23e1739a7671a1586d634b00803a1d19d 100644 (file)
@@ -813,10 +813,10 @@ drop table parted_conflict;
 -- partition
 create table parted_conflict (a int, b text) partition by range (a);
 create table parted_conflict_1 partition of parted_conflict for values from (0) to (1000) partition by range (a);
+create table parted_conflict_1_1 partition of parted_conflict_1 for values from (0) to (500);
 create unique index on only parted_conflict_1 (a);
 create unique index on only parted_conflict (a);
 alter index parted_conflict_a_idx attach partition parted_conflict_1_a_idx;
-create table parted_conflict_1_1 partition of parted_conflict_1 for values from (0) to (500);
 insert into parted_conflict values (40, 'forty');
 insert into parted_conflict_1 values (40, 'cuarenta')
   on conflict (a) do update set b = excluded.b;
index 961cffd20913ccf4106700507ff79c2ece8bd6de..43691cd3357985b758ecdb5e2e847a7262e00d58 100644 (file)
@@ -528,10 +528,10 @@ drop table parted_conflict;
 -- partition
 create table parted_conflict (a int, b text) partition by range (a);
 create table parted_conflict_1 partition of parted_conflict for values from (0) to (1000) partition by range (a);
+create table parted_conflict_1_1 partition of parted_conflict_1 for values from (0) to (500);
 create unique index on only parted_conflict_1 (a);
 create unique index on only parted_conflict (a);
 alter index parted_conflict_a_idx attach partition parted_conflict_1_a_idx;
-create table parted_conflict_1_1 partition of parted_conflict_1 for values from (0) to (500);
 insert into parted_conflict values (40, 'forty');
 insert into parted_conflict_1 values (40, 'cuarenta')
   on conflict (a) do update set b = excluded.b;