Re: elog() proposal

Peter Eisentraut wrote:
> Tom Lane writes:
> 
> > Peter Eisentraut <peter_e@gmx.net> writes:
> > > I think there should be separately named functions.
> >
> > You mean elog_debug(), elog_error(), ...?  Seems pretty yucky.
> 
> I would rather avoid the name "elog" altogether, for semantic and
> compatibility reasons.  We could invent much nicer names, e.g.,
> 
> LogDebug(int, const char*, ...)
> LogInfo(const char*, ...)
> AbortTransaction(const char*, ...)
> AbortSession(const char*, ...)
> AbortAllSessions(const char*, ...)

Wow, seems awfully complex to me.  Doesn't our current system seem
clearer:
elog(DEBUG, ...)elog(NOTICE, ...)elog(ERROR, ...)

Our elog messages are long enough already.

> > > and we don't have to tag all strings as translatable.
> >
> > Huh?  How does having multiple functions instead of one help there?
> 
> The string extractor can only go by function name, not arguments.

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 that "type of error" really conveys much, or that we can
> > always tell what the cause of an error report is.
> 
> 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.  I don't see the
value in tons of options.

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