Looks fine to me Tristan, I'll sponsor this change for you.
Trivially, I will add a 30sec timeout to the wait, just in case there is a problem in the future.
-Chris. On 11/04/2013 03:12 AM, Tristan Yan wrote:
Hi Everyone Please review the fix for JDK-8023462 <https://bugs.openjdk.java.net/browse/JDK-8023462>. Description: This test is using Thread.sleep(3000) to sync between server and client, this could be wrong when running is slow machine. I am trying to use modern CountDownLatch sync server and client. http://cr.openjdk.java.net/~pzhang/Tristan/8023462/webrev/ Thank you Tristan
