Patrick Hunt updated ZOOKEEPER-219:

    Attachment: ZOOKEEPER-219.patch

add resonable wait times for event polling.

In some cases the poll was expected to not return results (depending on args to 
the function) and in other cases it was expecting. I suspect the short poll 
times were to shorten the run time of the test (re no result expected). I moved 
to 10seconds to handle this, short enough to let the test run in resonable 
time, long enough to handle slowness in the system (cpu/disk loading outside 
zk, etc...)

> events.poll timeout in watcher test too short
> ---------------------------------------------
>                 Key: ZOOKEEPER-219
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-219
>             Project: Zookeeper
>          Issue Type: Bug
>    Affects Versions: 3.0.0
>            Reporter: Patrick Hunt
>            Priority: Minor
>             Fix For: 3.0.1, 3.1.0
>         Attachments: ZOOKEEPER-219.patch
> in watchertest there are some event.poll calls that have 1milli timeouts
>         e = localWatcher.events.poll(1, TimeUnit.MILLISECONDS);
> this is showing falure in some cases under hudson (I assume when it's under 
> load from other tests running for other proj)
> We should review the poll calls and verify adequate timeouts.

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