change reindex ERROR/NOTICE message

Author: Hiroshi Inoue

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();

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;
 				}