elog() proposals

Started by Bruce Momjianabout 24 years ago2 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

I just submitted a patch to fix various elog() issues. I have two
additional proposals.

First, I think ERROR/DEBUG/NOTICE/FATAL, etc are too generic and cause
compile problems/warnings, especially with Perl. I suggest renaming all
elog levels to PG*, so it would be PGERROR and PGINFO. We could also do
E_* or E*. I am interested in other opinions.

Second, I propose adding two GUC variables that control how much elog()
info is sent to the server and client logs. I suggest
'server_message_min' with possible values DEBUG, LOG, NOTICE, ERROR,
FATAL, CRASH; and 'client_message_min' with possible values INFO,
NOTICE, ERROR, FATAL, CRASH.

We currently have 'debug_level' which controls how much debug
information is sent to the server logs. I believe this would have to
remain because it controls how _much_ DEBUG output is printed. We could
go with some kind of hybrid where "DEBUG 5" sets DEBUG as the minimum
reporting mode with level 5.

This functionality mimics the log filter functionality of syslog(3).

Comments?

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
#2Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#1)
Re: elog() proposals

Bruce Momjian wrote:

I just submitted a patch to fix various elog() issues. I have two
additional proposals.

First, I think ERROR/DEBUG/NOTICE/FATAL, etc are too generic and cause
compile problems/warnings, especially with Perl. I suggest renaming all
elog levels to PG*, so it would be PGERROR and PGINFO. We could also do
E_* or E*. I am interested in other opinions.

I think we will need to rename symbols for 7.3 to PG_ or E_ or something
like that, but keep #define's in place with the old values through 7.3,
and urge people to update their code for 7.3, and remove the old symbols
in 7.4. We did this with LIMIT for 7.2.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026