[jira] Commented: (ZOOKEEPER-61) Address (remove) use of sleep(#) in client/server test cases.

["Patrick Hunt (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-61?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12609961#action_12609961 ] 

Patrick Hunt commented on ZOOKEEPER-61:
---------------------------------------

quorumtest failed on asf hudson today. This test passed in the previous CI run, and this run there are supposedly no changes to the source, seems related to this issue:

http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/lastBuild/testReport/org.apache.zookeeper.test/QuorumTest/testClientwithoutWatcherObj/

> Address (remove) use of sleep(#) in client/server test cases.
> -------------------------------------------------------------
>
>                 Key: ZOOKEEPER-61
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-61
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: build
>            Reporter: Patrick Hunt
>            Assignee: Benjamin Reed
>
> Many of the java client/server test cases use "sleep(x)" during initial rendezvous. This makes the tests slow, very brittle and they tend to fail occasionally during CI (depending on load). In particular I've seen QuorumTest exhibit this.
> Would also be nice if we could extract the "base" client/server test functionality to enable re-use and make test writing easier. I started this with ClientBase in tests but could do a better job.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.