[
https://issues.apache.org/jira/browse/ZOOKEEPER-1324?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13549219#comment-13549219
]
Flavio Junqueira commented on ZOOKEEPER-1324:
---------------------------------------------
It looks pretty good, Thawan, thanks for coming up with it fast.
I was planning on removing that ack in learner handler after receiving
UPTODATE. I believe it is there for backward compatibility with the 3.3 branch.
I was also wondering about the synchronization of startZKServer. That whole
block of code was previously synchronized in processAck, but if I'm reading it
right only lastCommitted can have concurrent accesses in that block, is it
right?
> Remove Duplicate NEWLEADER packets from the Leader to the Follower.
> -------------------------------------------------------------------
>
> Key: ZOOKEEPER-1324
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1324
> Project: ZooKeeper
> Issue Type: Bug
> Reporter: Mahadev konar
> Assignee: Thawan Kooburat
> Priority: Critical
> Fix For: 3.5.0
>
> Attachments: ZOOKEEPER-1324.patch, ZOOKEEPER-1324.patch,
> ZOOKEEPER-1324.patch, ZOOKEEPER-1324.patch
>
>
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira