[9] RFR(S): 8142464: PlatformLoggerTest.java throws java.lang.RuntimeException: Logger test.logger.bar does not exist

[Nils Eliasson]

Thank you Vladimir!

//Nils

On 2016-04-29 01:54, Vladimir Kozlov wrote:
> Good.
>
> thanks,
> Vladimir
>
> On 4/28/16 5:01 AM, Nils Eliasson wrote:
>> Hi,
>>
>> Please review the fix of 
>> jdk/test/sun/util/logging/PlatformLoggerTest.java that has been 
>> failing intermittently in our nightlies.
>>
>> Summary:
>> The test uses loggers that are accessible by weak refs from a 
>> LogManager. Since the test doesn't keep a strong ref to the loggers 
>> they may get collected during the test.
>>
>> Solution:
>> Save loggers to a static field  in the test class.
>>
>> Other:
>> Also removing "@compile -XDignore.symbol.file" that is unnecessary 
>> after Jigsaw. The compile tag will force a compile even if the class 
>> already exists, wasting times during reruns.
>>
>> Testing:
>> Running tests on all platforms.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8142464
>> webrev: http://cr.openjdk.java.net/~neliasso/8142464/webrev_jdk.01/ 
>> (JDK)
>>
>> Regards,
>> Nils Eliasson