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

Hongchao Deng commented on ZOOKEEPER-1559:
------------------------------------------

Is it good enough to append packets to the log instead of taking snapshot
{code}
    case Leader.NEWLEADER: // it will be NEWLEADER in v1.0
                    zk.takeSnapshot();
{code}

I think this is the easiest way to fix the problem. Any caveat in this?

> Learner should not snapshot uncommitted state
> ---------------------------------------------
>
>                 Key: ZOOKEEPER-1559
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1559
>             Project: ZooKeeper
>          Issue Type: Sub-task
>          Components: quorum
>            Reporter: Flavio Junqueira
>
> The code in Learner.java is a bit entangled for backward compatibility 
> reasons. We need to make sure that we can remove the calls to take a snapshot 
> without breaking it. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to