[
https://issues.apache.org/jira/browse/ZOOKEEPER-1638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13932701#comment-13932701
]
Hudson commented on ZOOKEEPER-1638:
-----------------------------------
FAILURE: Integrated in ZooKeeper-trunk #2247 (See
[https://builds.apache.org/job/ZooKeeper-trunk/2247/])
ZOOKEEPER-1638. Redundant zk.getZKDatabase().clear(); (neil bhakta via michim)
(michim:
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1576982)
* /zookeeper/trunk/CHANGES.txt
* /zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/Learner.java
> Redundant zk.getZKDatabase().clear();
> -------------------------------------
>
> Key: ZOOKEEPER-1638
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1638
> Project: ZooKeeper
> Issue Type: Improvement
> Reporter: Alexander Shraer
> Assignee: neil bhakta
> Priority: Trivial
> Labels: newbie
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-1638.patch, ZOOKEEPER-1638.patch
>
>
> Learner.syncWithLeader calls zk.getZKDatabase().clear() right before
> zk.getZKDatabase().deserializeSnapshot(leaderIs); Then the first thing
> deserializeSnapshot does is another clear().
> Suggest to remove the clear() in syncWithLeader.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
