This patch fixes the event type used to log output from the

stderr-in-service or output-from-syslogger-in-service code. Previously
everything was flagged as ERRORs there, which caused all instances to
log "LOG: logger shutting down" as error...

Please apply for 8.1. I'd also like it considered for 8.0 since logging
non-errors as errors can be cause for alarm amongst people who actually
look at their logs...

Magnus Hagander

diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c
index 2a26fe7fd409ffa4a9d795a5d8c64d4f58673294..62919ffb04c87fe36c268ffe387cfbe30b9bccf1 100644 (file)
--- a/src/backend/utils/error/elog.c
+++ b/src/backend/utils/error/elog.c
@@ -42,7 +42,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.161 2005/06/10 20:48:54 tgl Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/error/elog.c,v 1.162 2005/08/12 21:36:59 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1266,6 +1266,7 @@ write_syslog(int level, const char *line)
 static void
 write_eventlog(int level, const char *line)
 {
+       int eventlevel = EVENTLOG_ERROR_TYPE;
        static HANDLE evtHandle = INVALID_HANDLE_VALUE;
 
        if (evtHandle == INVALID_HANDLE_VALUE)
@@ -1278,8 +1279,33 @@ write_eventlog(int level, const char *line)
                }
        }
 
+       switch (level)
+       {
+               case DEBUG5:
+               case DEBUG4:
+               case DEBUG3:
+               case DEBUG2:
+               case DEBUG1:
+               case LOG:
+               case COMMERROR:
+               case INFO:
+               case NOTICE:
+                       eventlevel = EVENTLOG_INFORMATION_TYPE;
+                       break;
+               case WARNING:
+                       eventlevel = EVENTLOG_WARNING_TYPE;
+                       break;
+               case ERROR:
+               case FATAL:
+               case PANIC:
+               default:
+                       eventlevel = EVENTLOG_ERROR_TYPE;
+                       break;
+       }
+
+
        ReportEvent(evtHandle,
-                               level,
+                               eventlevel,
                                0,
                                0,                              /* All events are Id 0 */
                                NULL,
@@ -1634,32 +1660,7 @@ send_message_to_server_log(ErrorData *edata)
        /* Write to eventlog, if enabled */
        if (Log_destination & LOG_DESTINATION_EVENTLOG)
        {
-               int                     eventlog_level;
-
-               switch (edata->elevel)
-               {
-                       case DEBUG5:
-                       case DEBUG4:
-                       case DEBUG3:
-                       case DEBUG2:
-                       case DEBUG1:
-                       case LOG:
-                       case COMMERROR:
-                       case INFO:
-                       case NOTICE:
-                               eventlog_level = EVENTLOG_INFORMATION_TYPE;
-                               break;
-                       case WARNING:
-                               eventlog_level = EVENTLOG_WARNING_TYPE;
-                               break;
-                       case ERROR:
-                       case FATAL:
-                       case PANIC:
-                       default:
-                               eventlog_level = EVENTLOG_ERROR_TYPE;
-                               break;
-               }
-               write_eventlog(eventlog_level, buf.data);
+               write_eventlog(edata->elevel, buf.data);
        }
 #endif   /* WIN32 */
 
@@ -1675,7 +1676,7 @@ send_message_to_server_log(ErrorData *edata)
                 * because that's really a pipe to the syslogger process.
                 */
                if ((!Redirect_stderr || am_syslogger) && pgwin32_is_service())
-                       write_eventlog(EVENTLOG_ERROR_TYPE, buf.data);
+                       write_eventlog(edata->elevel, buf.data);
                else
 #endif
                        fprintf(stderr, "%s", buf.data);