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


Reply via email to