I don't think we (dev) have the resources ourselves to perform this testing, since testing this chane necessarily requires a wide variety of JVMs. It sounds like we have a bit of a chicken and egg problem here.
I made the change for these reasons: - No existing code relies on this change - It's not required - folks can still use logmanager.getlogger(classname) - It reuses existing (LocationInfo) code for class name discovery - It's a (very) convenient feature - We're in an alpha The last one may be controversial, but I feel we have to rely on the variety of configurations out there to verify the change works - if it doesn't work in some cases, we pull it or document it. But we should (at some point) at least try to provide this feature, obviously documenting it's possible limitations. I'm not arguing for a re-add, I'm just explaining why I made the commit. -----Original Message----- From: Ceki Gülcü [mailto:[EMAIL PROTECTED] Sent: Mon 12/20/2004 3:09 AM To: Log4J Developers List Cc: Subject: Re: cvs commit: logging-log4j/src/java/org/apache/log4j LogManager.java Scott, Please see http://marc.theaimsgroup.com/?t=110150364000001&r=1&w=2 Log4, as the most popular a logging API, must be reliable. Speed, convenience, user demand are all secondary considerations when compared with reliably. This change severely compromises reliability. It must be either backed up with substantial testing to ensure it is safe or it must go. When a change of this importance are done casually, it gets me worried. At 08:28 AM 12/20/2004, [EMAIL PROTECTED] wrote: >sdeboy 2004/12/19 23:28:07 > > Modified: src/java/org/apache/log4j LogManager.java > Log: > Added a no-arg getLogger method that returns a Logger using the > caller's class name (relies on the new LocationInfo classes). > -- Ceki Gülcü The complete log4j manual: http://qos.ch/log4j/ --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
<<winmail.dat>>
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
