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

Peter Eisentraut wrote:
> 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.

OK.  Glad it isn't a big issue.

> > > 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 need to renumber them.  It is backward compatible at a source
code level, not an object code level.  Is object code backward
compability for elog() an issue?  If so, I don't need to renumber them.

> > 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*.

Breaks object code only, which I think is minor, but I don't have to.

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

Tom and I came up with that one.  Feel free to suggest another.

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

Good.

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

So you want two spaces after every colon, no matter what?  Sure.  I just
makes the server logs jagged but it is a win on the user side.

--  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,
Pennsylvania19026