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

Flavio Junqueira commented on ZOOKEEPER-1113:
---------------------------------------------

Hi Alex, We tried to make the computation of containsQuorum as simple as 
possible to be efficient, but it is right that it has the drawback you point 
out. Also, we haven't measured the performance impact, so in reality, we don't 
know if having a loop and checking every vote would make any noticeable 
difference.

I checked that core test failure is not due to this patch. I'll be reviewing 
this patch shortly.

> QuorumMaj counts the number of ACKs but does not check who sent the ACK
> -----------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-1113
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1113
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: quorum
>    Affects Versions: 3.4.0
>            Reporter: Alexander Shraer
>            Priority: Minor
>         Attachments: ZOOKEEPER-1113.patch
>
>
> If a server connects to the leader as follower, it will be allowed to vote 
> (with QuorumMaj) even if it is not a follower in the current configuration,
> as the leader does not care who sends the ACK - it only counts the number of 
> ACKs.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to