logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Kevin Steppe <kste...@pacbell.net>
Subject Error Handler & Log4j JDBCAppender
Date Thu, 06 Jun 2002 17:47:06 GMT
I ran some tests to generate errors in flushBuffer() [lines 226-280]. 
 The first instance of errorHandler.error generated some console output. 
 The second instance (line 267) did not generate anything.  I did not 
specify any particular errorHandler.  When I placed a System.out.println 
inside that catch block I got it's output.  Thus it appears that by 
default the error call on line 267 will not generate output.  

Could you advise me or look at what is happening here?

Re: Bug --
No, the last JDBCAppender I sent you did not fix the bug.  I am working 
on that right now.

Ceki Gülcü wrote:

> At 09:28 06.06.2002 -0700, you wrote:
>> The double quote should fix #2.  Any time you are putting a string 
>> into a varchar type field you should use either single or double 
>> quotes. Without quotes you'll have all sorts of errors.  As for 
>> performance successful statements will always go faster than failed 
>> statements :)
>> The preparedStatement option seems nice in theory but I believe 
>> doesn't merit the work:
>> 1) The configuration would need to be more complicated to properly 
>> handle the variety of types/values/etc you might want to set.
>> 2) Escaping is still a problem, b/c the appender has to handle any 
>> number of quotes in a general fashion (unlike your example where you 
>> know exactly how many and where the quotes are)
>> 3) The performance value of preparedStatements is debate-able.
>> 4) Batch inserts make far more difference and will be included very 
>> soon (it's being tested)
> Kevin,
> I have very briefly looked at the new JDBCAppender. It seems that it
> still does not address bug #8529
> (http://nagoya.apache.org/bugzilla/show_bug.cgi?id=8529). It should be
> fixed soon or otherwise I will have to mark it (JDBCAppender) as
> unreliable.
> -- 
> Ceki
> Sign the petition: http://www.petitiononline.com/1234567b
> I am signatory number 22106. What is your number?
> -- 
> To unsubscribe, e-mail:   
> <mailto:log4j-user-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: 
> <mailto:log4j-user-help@jakarta.apache.org>

To unsubscribe, e-mail:   <mailto:log4j-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:log4j-dev-help@jakarta.apache.org>

View raw message