The patch updates the documentation to reflect the fact that higher values

of client_min_messages (fatal + panic) are valid and also fixes a slight
issue with how psql tried to display error messages that aren't sent to
the client.

We often tell people to ignore errors in response to requests for things
like "drop if exists", but there's no good way to completely hide this
without upping client_min_messages past ERROR.  When running a file like

SET client_min_messages TO 'FATAL';

DROP TABLE doesntexist;

with "psql -f filename" you get an error prefix of
"psql:/home/username/filename:3" even though there is no error message to
prefix because it isn't sent to the client.

Kris Jurka

diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 5da06fddce351a420aa46dad56270de608f3f5c3..b64ba6c1561190992e01a8ae9978cc414e8b2fee 100644 (file)
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -1,5 +1,5 @@
 
 
   Run-time Configuration
@@ -2295,7 +2295,8 @@ SELECT * FROM parent WHERE key = 2400;
         Valid values are DEBUG5,
         DEBUG4, DEBUG3, DEBUG2,
         DEBUG1, LOG, NOTICE,
-        WARNING, and ERROR.  Each level
+        WARNING, ERROR, FATAL,
+        and PANIC.  Each level
         includes all the levels that follow it.  The later the level,
         the fewer messages are sent.  The default is
         NOTICE.  Note that LOG has a different

diff --git a/src/bin/psql/common.c b/src/bin/psql/common.c
index 9b087adf01066ad65fa0e55760e5a47f7edb37ec..886bcde178635a0a0c56c4ba8fcff60fd42dc2e3 100644 (file)
--- a/src/bin/psql/common.c
+++ b/src/bin/psql/common.c
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2005, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/common.c,v 1.106 2005/10/04 19:01:18 petere Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/common.c,v 1.107 2005/10/13 20:58:42 momjian Exp $
  */
 #include "postgres_fe.h"
 #include "common.h"
@@ -681,7 +681,10 @@ AcceptResult(const PGresult *result, const char *query)
 
    if (!OK)
    {
-       psql_error("%s", PQerrorMessage(pset.db));
+       const char *error = PQerrorMessage(pset.db);
+       if (strlen(error))
+           psql_error("%s", error);
+
        ReportSyntaxErrorPosition(result, query);
        CheckConnection();
    }