RFR: 8023168 - Cleanup LogManager class initialization and LogManager/LoggerContext relationship

David M. Lloyd david.lloyd at redhat.com
Wed Sep 4 21:36:13 UTC 2013


On 09/04/2013 02:56 PM, Daniel Fuchs wrote:
> Hi,
>
> Please find below a changeset that will fix
>
> 8023168: Cleanup LogManager class initialization and
>           LogManager/LoggerContext relationship.
>
> <http://cr.openjdk.java.net/~dfuchs/webrev_8023168/webrev.00/>
>
> LogManager class initialization is fragile: because Logger
> constructor calls LogManager.getLogManager, and because
> LogManager calls new Logger during LogManager class initialization,
> and because the global logger is instantiated during Logger class
> initialization, loading Logger.class or LogManager.class can lead to
> challenging hard to diagnose issues.

As far as calling initialization ("ensureLogManagerInitialized()"), it's 
a shame that checking for a one-time action has to run through a 
synchronization block every time.  Maybe a "lazy holder" class would be 
more appropriate here, especially given that the point seems to be to 
produce the RootLogger() instance which doubles as the indicator that 
initialization was done.

There's a grammatical error in the comment at line 1547 of 
LogManager.java: "We do not the protected Logger <...>".

-- 
- DML



More information about the core-libs-dev mailing list