Disallow CLUSTER using an invalid index (that is, one left over from a failed

CREATE INDEX CONCURRENTLY).  Such an index might not have entries for every
heap row and thus clustering with it would result in silent data loss.
The scenario requires a pretty foolish DBA, but still ...

diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c
index 506e3d7894b8dc285d5ae242e6eb9d44ed9d2251..f0825a9d02fbb293389d375c8dcde32064cd6bf9 100644 (file)
--- a/src/backend/commands/cluster.c
+++ b/src/backend/commands/cluster.c
@@ -11,7 +11,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/commands/cluster.c,v 1.154.2.1 2007/09/10 22:02:05 alvherre Exp $
+ *       $PostgreSQL: pgsql/src/backend/commands/cluster.c,v 1.154.2.2 2007/09/29 18:05:28 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -417,6 +417,16 @@ check_index_is_clusterable(Relation OldHeap, Oid indexOid, bool recheck)
                                                        RelationGetRelationName(OldIndex))));
        }
 
+       /*
+        * Disallow if index is left over from a failed CREATE INDEX CONCURRENTLY;
+        * it might well not contain entries for every heap row.
+        */
+       if (!OldIndex->rd_index->indisvalid)
+               ereport(ERROR,
+                               (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                errmsg("cannot cluster on invalid index \"%s\"",
+                                               RelationGetRelationName(OldIndex))));
+
        /*
         * Disallow clustering system relations.  This will definitely NOT work
         * for shared relations (we have no way to update pg_class rows in other