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

Benjamin Reed commented on ZOOKEEPER-133:
-----------------------------------------

I'm thrown by:

 
+                // small chance that is hasn't been delivered yet, ignore
+                assertEquals(0, watchers[i].events.size());

Why would we ignore? If size > 0, it means that we got an event we weren't 
expecting right? The check looks correct, but the comment looks wrong. 



> hudson tests failing intermittently
> -----------------------------------
>
>                 Key: ZOOKEEPER-133
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-133
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: tests
>    Affects Versions: 3.0.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>             Fix For: 3.0.0
>
>         Attachments: ZOOKEEPER-133.patch
>
>
> There is a bug in the ClientTest.java unit test, timing issue in 
> "withWatcherObj" test. Patch forthcoming.

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

Reply via email to