RFR : JDK-8164609 : javax/management/remote/mandatory/notif/DeadListenerTest.java fails with AssertionError

[David Holmes]

Hi Amit,

On 29/08/2016 3:55 PM, Amit Sapre wrote:
> Hello,
>
> Please review a small test only change.
>
> Bug id : https://bugs.openjdk.java.net/browse/JDK-8164609
>
> Webrev : http://cr.openjdk.java.net/~sballal/sponsorship/8164609/webrev.00/

!  * Copyright (c) 2010, 2015, 2016, Oracle and/or its affiliates. All 
rights reserved.

Copyright format is wrong - we only use first and last year, so just 
update the existing 2015 to be 2016.

!  * @run main/othervm/timeout=120 -XX:+UsePerfData DeadListenerTest

I think that is already the default timeout. Only set an explicit 
timeout if you need it to be longer - which I don't think you do here.

Getting rid of the internal timeout mechanism is good.

> PS : Based on triaging , it looked like a timing issue in test case, so
> the proposed changes are done only to increase the test timeout. If
> issue is still observed, will re-open the bug.

Once a bug has had a changeset associated with it, it is considered 
Fixed and must not be reopened. Any subsequent test failures will 
require a new bug to be filed.

Thanks,
David

>
>
> Thanks,
>
> Amit
>