Jaroslav,
The fix should be good to fix the failure.
It makes me think a special case, suppose that the test waits 2
notifications, but the test might receive one unexpected notification
with some more waiting, for example, with the old version, 2 expected
notifications arrive within the first second, and the unexpected arrives
in the second second, but with your fix the test might end before the
unexpected notification arrives.
Not sure that we should take care of this case.
Thanks,
Shanliang
Jaroslav Bachorik wrote:
Please, review the following test change.
Issue : https://bugs.openjdk.java.net/browse/JDK-7132590
Webrev: http://cr.openjdk.java.net/~jbachorik/7132590/webrev.00
Currently, the test waits for an arbitrary time until it gives up on
receiving the notifications. This leads to intermittent failures in
situations when the execution is slower than anticipated (running
against a debug build etc.).
The solution is to block the test until all the expected notification
had been delivered or the test is timed out by the harness.
Thanks,
-JB-
