Re: elog() proposal - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: elog() proposal
Date
Msg-id Pine.LNX.4.30.0202222008140.686-100000@peter.localdomain
Whole thread Raw
In response to Re: elog() proposal  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: elog() proposal
Re: elog() proposal
Re: elog() proposal
List pgsql-hackers
Bruce Momjian writes:

> Can't we hack it to pull out only certain elogs()?  Also, don't we have
> to translate everything?  I guess not.

I'm not sure.  Someone other than me raised this point once.  It's not so
important.  I supposed, eventually people will want to translate
everything.  Feel free to keep it as once function.

> > What I mean with "type of error" is that there's a significant difference
> > between user errors  and server-side errors:
> >
> > 1. User errors should not necessarily go into the server log, unless
> > command logging is enabled.
> >
> > 2. User errors will eventually carry additional information such as error
> > codes.  Server errors will just get one default error code.
> >
> > 3. Users should not necessarily be allowed to see the details of server
> > errors at the client side, only some generic message.
> >
> > So if we made up two separate functions each for errors and notices, we
> > could raise the awareness about this, even if initially the functionality
> > would not differ much.
>
> Seems my solution is smaller and backward compatible.

Your solution renumbers the error codes, so it's definitely not
backward-compatible.

> I don't see the value in tons of options.

Well, I do.  We don't need the separate user-side error functions
initially, but eventually we will have to have them.

So, basically, what this comes down to with respect to your patch:

1. Renumbering the error codes breaks backward compatibility *silently*.

2. CRASH doesn't seem like a good name to me.

3. I agree with adding a LOG or INFO level between DEBUG and NOTICE.

4. I don't like the alignment change.  That seems very un-computer-like.

-- 
Peter Eisentraut   peter_e@gmx.net



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: elog() proposal
Next
From: Peter Eisentraut
Date:
Subject: Re: Going through email