[ https://issues.apache.org/jira/browse/HBASE-10988?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13972065#comment-13972065 ]
Hudson commented on HBASE-10988: -------------------------------- SUCCESS: Integrated in hbase-0.96 #387 (See [https://builds.apache.org/job/hbase-0.96/387/]) HBASE-10988 Addendum, break long line. (larsh: rev 1588020) * /hbase/branches/0.96/hbase-thrift/src/test/java/org/apache/hadoop/hbase/thrift/TestThriftServerCmdLine.java > Properly wait for server in TestThriftServerCmdLine > --------------------------------------------------- > > Key: HBASE-10988 > URL: https://issues.apache.org/jira/browse/HBASE-10988 > Project: HBase > Issue Type: Bug > Components: test > Reporter: Lars Hofhansl > Assignee: Lars Hofhansl > Priority: Minor > Fix For: 0.99.0, 0.94.19, 0.98.2, 0.96.3 > > Attachments: 10988-0.94.txt, 10988-trunk.txt > > > In 0.94 I find: > {{Threads.sleepWithoutInterrupt(2000)}} > In trunk I see: > {code} > while ( thriftServer.serverRunner == null || > thriftServer.serverRunner.tserver == null ){ > Thread.sleep(1); > } > {code} > Both aren't good. > The 0.94 version will fail if the server does not come up within 2s. The > trunk version (1) might wait forever and cause a long timeout for the test > and (2) wait quite busily with only 1ms of sleeping. -- This message was sent by Atlassian JIRA (v6.2#6252)