https://issues.apache.org/bugzilla/show_bug.cgi?id=50686
Rainer Jung <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #2 from Rainer Jung <[email protected]> 2011-01-30 06:04:33 EST --- Hi Milamber, thanks for downgrading the message. Please reconsider merging the two debug log statements: - The missing isDebugEnabled() for the first call makes it more expensive than necessary. The whole argument formatting will be done before calling debug() for every sample, even if debug logging is not activated. - The two debug messages are redundant and joining them into one message reduces debug log volume without removing any information from the log Thanks again! Rainer -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
