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

[Daniil Titov]

Hi David and Alex,

Thank for reviewing this change.

I will push it to jdk15 repo as David suggested.

Best regards,
Daniil

On 6/11/20, 5:28 PM, "Alex Menkov" <alexey.menkov at oracle.com> 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
    >>
    >>
    >>