Hi Amit,

Increasing the timeout 'TIME_OUT_FACTOR' will increase both socket and test-case timeout. The test passed as can be seen from the log, but because of the race-condition: hasTestLivedLongEnough() checked before shouldContinue(), the test was declared failed because of time-out.

One possible fix would be to change line 80 to,

                if (shouldContinue() && hasTestLivedLongEnough())

Thanks

Harsha


On Wednesday 13 December 2017 11:03 AM, Amit Sapre wrote:

Hello,

Please review the timeout fix for this bug.

Bug ID : https://bugs.openjdk.java.net/browse/JDK-8175542

Webrev : http://cr.openjdk.java.net/~asapre/webrev/2017/JDK-8175542/webrev.00/ <http://cr.openjdk.java.net/%7Easapre/webrev/2017/JDK-8175542/webrev.00/>

Thanks,

Amit


Reply via email to