RFR : JDK-7132577 - javax/management/monitor/MultiMonitorTest.java fails in JDK8-B22

Harsha Wardhana B harsha.wardhana.b at oracle.com
Wed Feb 22 14:18:29 UTC 2017


Hi Amit,

There is no need to wait in a loop to check we have not received any 
notifications. Without starting the monitors, the listener count will be 
zero.

The first part of diff L116-L121 could be left as is.

By relying on Jtreg timeout for receiving notifications, we will be 
unable to print the number of listeners emitted by each Monitor. But I 
guess there is no way to intercept a Jtreg timeout and print out those 
values.

Regards

Harsha


On Wednesday 22 February 2017 03:29 PM, Amit Sapre wrote:
>
> Hello,
>
> Please review this test bug fix which eliminates test case’s own 
>  timeout mechanism to default jtreg timeout.
>
> Bug ID : https://bugs.openjdk.java.net/browse/JDK-7132577
>
> Webrev : 
> http://cr.openjdk.java.net/~asapre/webrev/2017/JDK-7132577/webrev.00/ 
> <http://cr.openjdk.java.net/%7Easapre/webrev/2017/JDK-7132577/webrev.00/>
>
> Thanks,
>
> Amit
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20170222/a0c8a693/attachment.html>


More information about the serviceability-dev mailing list