[ https://issues.apache.org/jira/browse/ZOOKEEPER-2137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320779#comment-15320779 ]
Michael Han commented on ZOOKEEPER-2137: ---------------------------------------- Yup I think you are right [~rgs], I could see one case where an existing writer node created by a different call of testNormalOperation (where the zk handles were shared between calls) can throw the exception that skip reader node creation - which might be the cause of the failure in previous pre-commit tests. > Make testPortChange() less flaky > -------------------------------- > > Key: ZOOKEEPER-2137 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2137 > Project: ZooKeeper > Issue Type: Sub-task > Reporter: Hongchao Deng > Assignee: Michael Han > Fix For: 3.5.2, 3.6.0 > > Attachments: ZOOKEEPER-2137-cb.patch, ZOOKEEPER-2137.patch, > ZOOKEEPER-2137.patch, ZOOKEEPER-2137.patch > > > The cause of flaky failure of testPortChange() is a race in sync(). > I figured out it could take some time to fix sync(). Meanwhile, we can make > testPortChange() less flaky by doing reconfig on the leader. We can change > this back in the fix of ZOOKEEPER-2136. -- This message was sent by Atlassian JIRA (v6.3.4#6332)