You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2009/07/14 20:33:14 UTC
[jira] Commented: (ZOOKEEPER-460) bad testRetry in cppunit tests
(hudson failure)
[ https://issues.apache.org/jira/browse/ZOOKEEPER-460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12731026#action_12731026 ]
Patrick Hunt commented on ZOOKEEPER-460:
----------------------------------------
the build has been failing for the past 6 days, this is very bad -- in effect no CI
Henry, can you look at this or should I?
> bad testRetry in cppunit tests (hudson failure)
> -----------------------------------------------
>
> Key: ZOOKEEPER-460
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-460
> Project: Zookeeper
> Issue Type: Bug
> Components: c client, tests
> Reporter: Patrick Hunt
> Assignee: Henry Robinson
> Fix For: 3.2.1, 3.3.0
>
>
> the followng code failed on hudson
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/ZooKeeper-trunk/371/
> watchctx_t ctx1, ctx2;
> zhandle_t *zk1 = createClient(&ctx1);
> CPPUNIT_ASSERT_EQUAL(true, ctx1.waitForConnected(zk1));
> zhandle_t *zk2 = createClient(&ctx2);
> zookeeper_close(zk1);
> CPPUNIT_ASSERT_EQUAL(true, ctx2.waitForConnected(zk2));
> there's a problem with this test, it assumes that close(1) can be called before createclient(2) gets connected.
> this is not correct: createclient is an async call an in some cases the connection can be established before
> create client returns.
> this shows a failure in this case because client1 was created, then client2 attempted to connect
> but failed due to this on the server (max conn exceeded):
> sprintf(cmd, "export ZKMAXCNXNS=1;%s startClean %s", ZKSERVER_CMD, getHostPorts());
> conn 2 failed and therefore the following assert eventually failed.
> this code should not assume that close(1) will beat connect(2)
> Henry can you take a look?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.