diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-04-04 12:32:42 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-04-04 12:32:42 -0400 |
| commit | 66229ac0040cf1e0f5b9d72271aa9feaf3b3a37e (patch) | |
| tree | b2599d2bc1f7406150b65f9fbfe8c960c3527a4f /src/backend/utils/error/elog.c | |
| parent | 58666ed28ab59a2686ee08bc648b4e9959aacfce (diff) | |
| download | postgresql-66229ac0040cf1e0f5b9d72271aa9feaf3b3a37e.tar.gz postgresql-66229ac0040cf1e0f5b9d72271aa9feaf3b3a37e.zip | |
Introduce a LOG_SERVER_ONLY ereport level, which is never sent to client.
This elevel is useful for logging audit messages and similar information
that should not be passed to the client. It's equivalent to LOG in terms
of decisions about logging priority in the postmaster log, but messages
with this elevel will never be sent to the client.
In the current implementation, it's just an alias for the longstanding
COMMERROR elevel (or more accurately, we've made COMMERROR an alias for
this). At some point it might be interesting to allow a LOG_ONLY flag to
be attached to any elevel, but that would be considerably more complicated,
and it's not clear there's enough use-cases to justify the extra work.
For now, let's just take the easy 90% solution.
David Steele, reviewed by Fabien Coelho, Petr Jelínek, and myself
Diffstat (limited to 'src/backend/utils/error/elog.c')
| -rw-r--r-- | src/backend/utils/error/elog.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c index 8e006097a64..740f089ccdb 100644 --- a/src/backend/utils/error/elog.c +++ b/src/backend/utils/error/elog.c @@ -293,7 +293,7 @@ errstart(int elevel, const char *filename, int lineno, output_to_server = is_log_level_output(elevel, log_min_messages); /* Determine whether message is enabled for client output */ - if (whereToSendOutput == DestRemote && elevel != COMMERROR) + if (whereToSendOutput == DestRemote && elevel != LOG_SERVER_ONLY) { /* * client_min_messages is honored only after we complete the @@ -2086,7 +2086,7 @@ write_eventlog(int level, const char *line, int len) case DEBUG2: case DEBUG1: case LOG: - case COMMERROR: + case LOG_SERVER_ONLY: case INFO: case NOTICE: eventlevel = EVENTLOG_INFORMATION_TYPE; @@ -2965,7 +2965,7 @@ send_message_to_server_log(ErrorData *edata) syslog_level = LOG_DEBUG; break; case LOG: - case COMMERROR: + case LOG_SERVER_ONLY: case INFO: syslog_level = LOG_INFO; break; @@ -3595,7 +3595,7 @@ error_severity(int elevel) prefix = _("DEBUG"); break; case LOG: - case COMMERROR: + case LOG_SERVER_ONLY: prefix = _("LOG"); break; case INFO: @@ -3699,7 +3699,7 @@ write_stderr(const char *fmt,...) static bool is_log_level_output(int elevel, int log_min_level) { - if (elevel == LOG || elevel == COMMERROR) + if (elevel == LOG || elevel == LOG_SERVER_ONLY) { if (log_min_level == LOG || log_min_level <= ERROR) return true; |