On 17/01/2014 05:40, Tristan Yan wrote:
Hi All
Please review the simple code fix for JDK-8031666.
http://cr.openjdk.java.net/~tyan/JDK-8031666/webrev.00/
This test fails in very small chance. Adding socket timeout a little
bigger.
Looking at JDK-8031666 then the failure seems to be with the runAfter +
receive at end of test2. So I wonder have you the right issue number? In
the webrev then you've increased the timeout for tests that expect to
timeout (so I assume making the timeout longer just makes the test take
longer).
As regards the runAfter + receive at the end then maybe the issue is
that the runner thread was just slow to start, causing it to take more
than 3 seconds (which I think is tolerance of 2s in checkTime).
-Alan
