Re: logging output

At 10:31 PM 2/1/2004 -0800, you wrote:
>Hi all,
>
>I'll try to debug my problem and give you more infos about what's happening.
>Hopefully I can figure out what's going on.

Better make it quick if you want to get a change in for 1.2.6.  Otherwise, 
if you find it, it can always be fixed post-1.2.6 and you can use a CVS 
build until 1.2.7.

>In the meantime, there's one thing that bugs me when I look at my server
>output:  the number of info statements of Barracuda.  Really, for a simple
>page I maybe have 60/70 lines in my log.  When writing to the file/console
>can be very costly, I think that some precious ms can be saved by moving
>these info statements into debug ones.  And not forgetting the if
>(log.isDebugEnabled()).

Please provide examples of what info statements you see as being more 
appropriate as debug statements.  And most places in Barracuda use 
logger.isDebugEnabled() and logger.isInfoEnabled() around debug and info 
statements.  If there are places that don't do this, they should.  Patches 
are welcome.

>Of course, I can set the log level to Info but IMO Info statement are worth
>being logged on a production environment, when debug ones clearly are not..

Yes, but I wouldn't run at "info" level in production.  I would use warn, 
generally.  Info was always meant to be relatively verbose.  Keep in mind 
that 60/70 lines per page call for info is probably about right, since 
debug can be hundreds to thousands.  You can just put everything in 
debug.  Again, examples of what you think is inappropriate for "info" would 
be helpful.

>But it's just my opinion.

Opinions are welcome here, but in order to give them any weight, they must 
be backed up by some evidence.  I'm not saying you don't have any, just 
that none is provided in your email.  If you provide it, the issue will be 
more likely to become an action item.

Jake

>-Vladimir
>
>__________________________________
>Do you Yahoo!?
>Yahoo! SiteBuilder - Free web site building tool. Try it!
>http://webhosting.yahoo.com/ps/sb/
>_______________________________________________
>Barracuda mailing list
>[email protected]
>http://barracudamvc.org/lists/listinfo/barracuda