hadoop-zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Steven Cheng (JIRA)" <j...@apache.org>
Subject [jira] Commented: (ZOOKEEPER-622) Test for pending watches in send_set_watches should be moved
Date Tue, 23 Feb 2010 08:33:27 GMT

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-622?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12837139#action_12837139

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.

View raw message