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

Flavio Junqueira commented on ZOOKEEPER-882:
--------------------------------------------

(I meant to post a comment yesterday, but jira decided to re-index right at the 
time)

I like the way you structured the restore loop, it is simpler and easier to 
read, and I can't find any problem with it. About the severity of the bug, my 
interpretation is that it is harmless to re-execute the transaction, but still 
worth proposing a patch.

> Startup loads last transaction from snapshot
> --------------------------------------------
>
>                 Key: ZOOKEEPER-882
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-882
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: server
>            Reporter: Jared Cantwell
>            Priority: Minor
>         Attachments: 882.diff, restore
>
>
> On startup, the server first loads the latest snapshot, and then loads from 
> the log starting at the last transaction in the snapshot.  It should begin 
> from one past that last transaction in the log.  I will attach a possible 
> patch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to