RFR: 8246196: javax/management/MBeanServer/OldMBeanServerTest fails with AssertionError

[serguei.spitsyn at oracle.com]

+1

Thanks,
Serguei


On 6/11/20 17:28, Alex Menkov wrote:
> +1
>
> --alex
>
> On 06/11/2020 16:51, David Holmes wrote:
>> Hi Daniil,
>>
>> On 12/06/2020 5:56 am, Daniil Titov wrote:
>>> Please review change [1] that fixes an intermittent  failure of the 
>>> test when it is runs with -Xcomp.
>>>
>>> The problem here is that the timespan the test uses to count 
>>> notifications  is not adjusted for "test.timeout.factor" system 
>>> property.
>>
>> The adjustment looks fine.
>>
>>> The original issue is reproducible in JDK 11 and on Solaris platform 
>>> only. However,  I think it makes sense to apply this change in JDK 
>>> 15 to prevent this from possible happening in the future and then 
>>> backport it to 11.
>>
>> Do you still intend this for 15 or just 16? If 15 then push to jdk15 
>> repo and it will get forward ported to jdk/jdk automatically.
>>
>> Thanks,
>> David
>>
>>> [1] http://cr.openjdk.java.net/~dtitov/8246196/webrev.01/
>>> [2] https://bugs.openjdk.java.net/browse/JDK-8246196
>>>
>>> Thank you,
>>> Daniil
>>>
>>>
>>>