Author: larsh
Date: Wed Apr 16 18:24:45 2014
New Revision: 1588016

URL: http://svn.apache.org/r1588016
Log:
HBASE-10988 Properly wait for server in TestThriftServerCmdLine.

Modified:
    
hbase/trunk/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java

Modified: 
hbase/trunk/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java?rev=1588016&r1=1588015&r2=1588016&view=diff
==============================================================================
--- 
hbase/trunk/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java
 (original)
+++ 
hbase/trunk/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java
 Wed Apr 16 18:24:45 2014
@@ -174,8 +174,9 @@ public class TestThriftServerCmdLine {
     thriftServer = new ThriftServer(TEST_UTIL.getConfiguration());
     startCmdLineThread(args.toArray(new String[0]));
 
-    while ( thriftServer.serverRunner == null || 
thriftServer.serverRunner.tserver == null ){
-      Thread.sleep(1);
+    // wait up to 10s for the server to start
+    for (int i=0; i<100 && (thriftServer.serverRunner == null || 
thriftServer.serverRunner.tserver == null); i++) {
+      Thread.sleep(100);
     }
 
     Class<? extends TServer> expectedClass = implType != null ?


Reply via email to