Steven Cheng commented on ZOOKEEPER-622:

This patch works but is still in rough shape!  The multi host test servers are 
started on separate ports and are running concurrently with the single host 
test server.  Would be helpful to make this more clear.

Paths still hardcoded to /tmp.

zooX.cfg are in the patch but are generated by generateCfgs.sh

I encountered a ZCONNECTIONLOSS from an operation executed after the client 
reconnects to a new host.  Adds an awkward section to the test case.  I guess 
in some sense the client should not be oblivious to one of these "lucky" 
disconnect/reconnect situations.

> Test for pending watches in send_set_watches should be moved
> ------------------------------------------------------------
>                 Key: ZOOKEEPER-622
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-622
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>            Reporter: Steven Cheng
>            Assignee: Benjamin Reed
>             Fix For: 3.3.0
>         Attachments: ZOOKEEPER-622.patch, ZOOKEEPER-622.patch, 
> ZOOKEEPER-622.patch
> Valgrind found:
> {quote}
> ==2357== Conditional jump or move depends on uninitialised value(s)
> ==2357==    at 0x807FDCA: check_events (zookeeper.c:1180)
> ==2357==    by 0x808043A: zookeeper_process (zookeeper.c:1775)
> ==2357==    by 0x806A21B: Zookeeper_close::testCloseConnected1() 
> (TestZookeeperClose.cc:161)
> ==2357==    by 0x806C6BF: CppUnit::TestCaller<Zookeeper_close>::runTest() 
> (TestCaller.h:166)
> {quote}
> zookeeper.c:1180 was the first if in send_set_watches.

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