Hi Daniel,
Thank you for reviewing the fix! That's what 'vm.heapHisto("-live")'
does - it will request a full GC.
Best regards,
Katja
On 12/16/2014 06:37 PM, Daniel Fuchs wrote:
Hi Katja,
Nice too see some more shell scripts go away!
I wonder - shouldn't the test force a System.gc()
at some point before taking the histogram?
If not what guarantee do we have that the various weak references
will be purged?
best regards,
-- daniel
On 16/12/14 16:57, Yekaterina Kantserova wrote:
Hi,
Could I please have a review of this fix.
bug: https://bugs.openjdk.java.net/browse/JDK-6977426
webrev: http://cr.openjdk.java.net/~ykantser/6977426/webrev.00/
java/util/logging/LoggerWeakRefLeak.sh and
java/util/logging/AnonLoggerWeakRefLeak.sh are merged and rewritten in
java. sun/tools/common/CommonTests.sh is removed because it doesn't test
the product but sun/tool/common library functionality.
Thanks,
Katja