[ https://issues.apache.org/jira/browse/ZOOKEEPER-1638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13788293#comment-13788293 ]
Raul Gutierrez Segales commented on ZOOKEEPER-1638: --------------------------------------------------- [~neilb]: did you upload the right patch? I think you wanted: {noformat} - // clear our own database and read + // db is clear as part of deserializeSnapshot() - zk.getZKDatabase().clear(); {noformat} > 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 > > > 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.1#6144)