Reindex of shared system indexes must be overwrite mode.

diff --git a/src/backend/catalog/index.c b/src/backend/catalog/index.c
index 1c2d6324cfc24431bdb09ae5d7065208813600ba..4d35ddeee8faed81c31b49bea5633ce555d34ab1 100644 (file)
--- a/src/backend/catalog/index.c
+++ b/src/backend/catalog/index.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.140 2001/01/29 00:39:16 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.141 2001/02/23 09:26:14 inoue Exp $
  *
  *
  * INTERFACE ROUTINES
@@ -2087,7 +2087,12 @@ reindex_index(Oid indexId, bool force, bool inplace)
 
 #ifndef OLD_FILE_NAMING
    if (!inplace)
-       setNewRelfilenode(iRel);
+        {
+                inplace = IsSharedSystemRelationName(NameStr(iRel->rd_rel->relna
+me));
+       if (!inplace)
+           setNewRelfilenode(iRel);
+   }
 #endif /* OLD_FILE_NAMING */
    /* Obtain exclusive lock on it, just to be sure */
    LockRelation(iRel, AccessExclusiveLock);
@@ -2196,6 +2201,20 @@ reindex_relation(Oid relid, bool force)
            elog(ERROR, "the target relation %u is nailed", relid);
    }
 #endif /* ENABLE_REINDEX_NAILED_RELATIONS */
+   /*
+    * Shared system indexes must be overwritten because it's
+    * impossible to update pg_class tuples of all databases.
+    */
+   if (IsSharedSystemRelationName(NameStr(rel->rd_rel->relname)))
+   {
+       if (IsIgnoringSystemIndexes())
+       {
+           overwrite = true;
+           deactivate_needed = true;
+       }
+       else
+           elog(ERROR, "the target relation %u is shared", relid);
+   }
    RelationClose(rel);
 #endif /* OLD_FILE_NAMING */
    old = SetReindexProcessing(true);

diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c
index 173c3a3d551db4b0c8b082b8d9a216f0f80a0cda..6497da615b8425f0360086607f8207bbca17d702 100644 (file)
--- a/src/backend/commands/indexcmds.c
+++ b/src/backend/commands/indexcmds.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/commands/indexcmds.c,v 1.44 2001/01/24 19:42:52 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/commands/indexcmds.c,v 1.45 2001/02/23 09:26:14 inoue Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -633,6 +633,7 @@ void
 ReindexIndex(const char *name, bool force /* currently unused */ )
 {
    HeapTuple   tuple;
+   bool        overwrite = false;
 
    /* ----------------
     *  REINDEX within a transaction block is dangerous, because
@@ -656,7 +657,9 @@ ReindexIndex(const char *name, bool force /* currently unused */ )
 #ifdef OLD_FILE_NAMING
    if (!reindex_index(tuple->t_data->t_oid, force, false))
 #else
-   if (!reindex_index(tuple->t_data->t_oid, force, false))
+   if (IsIgnoringSystemIndexes())
+       overwrite = true;
+   if (!reindex_index(tuple->t_data->t_oid, force, overwrite))
 #endif /* OLD_FILE_NAMING */
        elog(NOTICE, "index \"%s\" wasn't reindexed", name);