Code Review for WeakReference leak in the Logging API (6942989)
David Holmes
David.Holmes at oracle.com
Thu Jun 10 18:08:10 PDT 2010
Hi Dan,
I'm surprised to see that the VM team "owns" this one :)
Daniel D. Daugherty said the following on 06/11/10 03:13:
> I need a couple of code reviews for my fix for the WeakReference leak
> in the Logging API. The webrev is relative to OpenJDK7, but the bug
> is escalated so the fix will be backported to the JDK6-Update train.
> That's why I need at least two code reviewers.
>
> Here is the URL for the webrev:
>
> http://cr.openjdk.java.net/~dcubed/6942989-webrev/0/
I looked at the code and I read the CR and I understand what you are
doing - and in that sense it seems ok.
It does seem very complicated (but the logger management is complicated
:( ).
I wonder about the performance implications of doing this search each
time a logger is added? These types of cleanups are always a fine line
between minimizing cleanup up on the main path, and ensuring too much
garbage doesn't accumulate. I'm a little concerned this is a lot of
clean up code (potentially) on the main path.
David
More information about the serviceability-dev
mailing list