> I used wanted to point out the the ( -q, --quiet ) parameter for > reindexdb command utility does not work. Actually it is *not* a bug. The NOTICE is printed by the REINDEX command; reindexdb is just a wrapper around REINDEX command. If you set up 'client_min_messages' in postgresql.conf, you don't get those 'annoying' messages.
> Also reindexdb writes NOTICE to standard err for every table it > reindexes which makes it really hard to use in script as it always > produces an "errror". :-) > You can always ignore NOTICE. grep is your friend. :-) Looking at the code, IMHO we could raise the ereport from NOTICE to INFO (see the patch attached). None of the scripts actually raise NOTICEs if it succeeds. Comments? -- Euler Taveira de Oliveira http://www.timbira.com/
Index: src/backend/commands/indexcmds.c =================================================================== RCS file: /projects/cvsroot/pgsql/src/backend/commands/indexcmds.c,v retrieving revision 1.139 diff -c -r1.139 indexcmds.c *** src/backend/commands/indexcmds.c 10 May 2006 23:18:39 -0000 1.139 --- src/backend/commands/indexcmds.c 31 May 2006 01:28:49 -0000 *************** *** 1108,1114 **** /* functions in indexes may want a snapshot set */ ActiveSnapshot = CopySnapshot(GetTransactionSnapshot()); if (reindex_relation(relid, true)) ! ereport(NOTICE, (errmsg("table \"%s\" was reindexed", get_rel_name(relid)))); CommitTransactionCommand(); --- 1108,1114 ---- /* functions in indexes may want a snapshot set */ ActiveSnapshot = CopySnapshot(GetTransactionSnapshot()); if (reindex_relation(relid, true)) ! ereport(INFO, (errmsg("table \"%s\" was reindexed", get_rel_name(relid)))); CommitTransactionCommand();
---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings