diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2008-02-20 14:31:35 +0000 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2008-02-20 14:31:35 +0000 |
| commit | f78611bba4b9d5a55429812014b7f256e72e5b5d (patch) | |
| tree | 7eb9f4b88342093e25f767bce669203381cc17dd /src/backend/commands/analyze.c | |
| parent | bccc8e36084f2ed69d172e064c30039e464fb07f (diff) | |
Improve error messages emitted when VACUUM and ANALYZE skip a table.
Per gripe from Clodoaldo Pinto Neto on
Message-ID: <a595de7a0801060326qbfc790ax2a60573043c2e2be@mail.gmail.com>
Diffstat (limited to 'src/backend/commands/analyze.c')
| -rw-r--r-- | src/backend/commands/analyze.c | 20 |
1 files changed, 16 insertions, 4 deletions
diff --git a/src/backend/commands/analyze.c b/src/backend/commands/analyze.c index 82bb020bc57..4d2ea3c6126 100644 --- a/src/backend/commands/analyze.c +++ b/src/backend/commands/analyze.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/commands/analyze.c,v 1.114 2008/01/03 21:23:15 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/commands/analyze.c,v 1.115 2008/02/20 14:31:35 alvherre Exp $ * *------------------------------------------------------------------------- */ @@ -22,6 +22,7 @@ #include "catalog/index.h" #include "catalog/indexing.h" #include "catalog/namespace.h" +#include "catalog/pg_namespace.h" #include "commands/dbcommands.h" #include "commands/vacuum.h" #include "executor/executor.h" @@ -161,9 +162,20 @@ analyze_rel(Oid relid, VacuumStmt *vacstmt, { /* No need for a WARNING if we already complained during VACUUM */ if (!vacstmt->vacuum) - ereport(WARNING, - (errmsg("skipping \"%s\" --- only table or database owner can analyze it", - RelationGetRelationName(onerel)))); + { + if (onerel->rd_rel->relisshared) + ereport(WARNING, + (errmsg("skipping \"%s\" --- only superuser can analyze it", + RelationGetRelationName(onerel)))); + else if (onerel->rd_rel->relnamespace == PG_CATALOG_NAMESPACE) + ereport(WARNING, + (errmsg("skipping \"%s\" --- only superuser or database owner can analyze it", + RelationGetRelationName(onerel)))); + else + ereport(WARNING, + (errmsg("skipping \"%s\" --- only table or database owner can analyze it", + RelationGetRelationName(onerel)))); + } relation_close(onerel, ShareUpdateExclusiveLock); return; } |