change reindex ERROR/NOTICE message

diff --git a/src/backend/commands/indexcmds.c b/src/backend/commands/indexcmds.c
index 19ff9597d5297129ce761b5161a77e8214fc4bdc..16ad849d8ffa81aadcec9fcb81ecc1f6267616f8 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.25 2000/04/25 02:45:54 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/commands/indexcmds.c,v 1.26 2000/04/25 10:38:38 inoue Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -714,7 +714,8 @@ ReindexIndex(const char *name, bool force /* currently unused */ )
             ((Form_pg_class) GETSTRUCT(tuple))->relkind);
    }
 
-   reindex_index(tuple->t_data->t_oid, force);
+   if (!reindex_index(tuple->t_data->t_oid, force))
+       elog(NOTICE, "index '%s' wasn't reindexed", name);
 }
 
 /*
@@ -744,7 +745,8 @@ ReindexTable(const char *name, bool force)
             ((Form_pg_class) GETSTRUCT(tuple))->relkind);
    }
 
-   reindex_relation(tuple->t_data->t_oid, force);
+   if (!reindex_relation(tuple->t_data->t_oid, force))
+       elog(NOTICE, "table '%s' wasn't reindexed", name);
 }
 
 /*
@@ -806,7 +808,6 @@ ReindexDatabase(const char *dbname, bool force, bool all)
        elog(ERROR, "REINDEX DATABASE: Can be executed only on the currently open database.");
 
    heap_close(relation, NoLock);
-   /** reindex_database(db_id, force, !all); **/
 
    CommonSpecialPortalOpen();
    pmem = CommonSpecialPortalGetMemory();
@@ -847,7 +848,8 @@ ReindexDatabase(const char *dbname, bool force, bool all)
    for (i = 0; i < relcnt; i++)
    {
        StartTransactionCommand();
-       reindex_relation(relids[i], force);
+       if (reindex_relation(relids[i], force))
+           elog(NOTICE, "relation %d was reindexed", relids[i]);
        CommitTransactionCommand();
    }
    CommonSpecialPortalClose();

diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c
index 2cc10fe1cd9781b64d3ccfae2b98b3d78169f6f2..89ac3a635ac588df2819734d555d71101697db77 100644 (file)
--- a/src/backend/tcop/utility.c
+++ b/src/backend/tcop/utility.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.86 2000/04/12 17:15:43 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.87 2000/04/25 10:38:38 inoue Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -865,11 +865,11 @@ ProcessUtility(Node *parsetree,
                        if (IsSystemRelationName(relname))
                        {
                            if (!allowSystemTableMods && IsSystemRelationName(relname))
-                               elog(ERROR, "class \"%s\" is a system catalog index",
-                                    relname);
+                               elog(ERROR, "\"%s\" is a system index. call REINDEX under standalone postgres with -O -P options",
+                                relname);
                            if (!IsIgnoringSystemIndexes())
-                               elog(ERROR, "class \"%s\" is a system catalog index",
-                                    relname);
+                               elog(ERROR, "\"%s\" is a system index. call REINDEX under standalone postgres with -P -O options",
+                                relname);
                        }
 #ifndef NO_SECURITY
                        if (!pg_ownercheck(userName, relname, RELNAME))
@@ -882,11 +882,12 @@ ProcessUtility(Node *parsetree,
                        if (IsSystemRelationName(relname))
                        {
                            if (!allowSystemTableMods && IsSystemRelationName(relname))
-                               elog(ERROR, "class \"%s\" is a system catalog index",
-                                    relname);
+                               elog(ERROR, "\"%s\" is a system table. call REINDEX under standalone postgres with -O -P options",
+                                relname);
                            if (!IsIgnoringSystemIndexes())
-                               elog(ERROR, "class \"%s\" is a system catalog index",
-                                    relname);
+                               elog(ERROR, "\"%s\" is a system table. call REINDEX under standalone postgres with -P -O options",
+
+                                relname);
                        }
 #ifndef NO_SECURITY
                        if (!pg_ownercheck(userName, relname, RELNAME))
@@ -897,9 +898,9 @@ ProcessUtility(Node *parsetree,
                    case DATABASE:
                        relname = (char *) stmt->name;
                        if (!allowSystemTableMods)
-                           elog(ERROR, "-O option is needed");
+                           elog(ERROR, "must be called under standalone postgres with -O -P options");
                        if (!IsIgnoringSystemIndexes())
-                           elog(ERROR, "-P option is needed");
+                           elog(ERROR, "must be called under standalone postgres with -P -O options");
                        ReindexDatabase(relname, stmt->force, false);
                        break;
                }