You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by Alexander Shraer <sh...@yahoo-inc.com> on 2014/04/13 03:22:39 UTC

Review Request 20287: Leader should timeout when loosing quorum of either old or new config

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20287/
-----------------------------------------------------------

Review request for zookeeper.


Bugs: zookeeper-1699
    https://issues.apache.org/jira/browse/zookeeper-1699


Repository: zookeeper


Description
-------

Change the way syncSet is calculated in leader.java, to account for the proposed config, which also should have a quorum.
Leader should time out when loosing quorum of either old or newly proposed config.


Diffs
-----

  ./src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1586860 

Diff: https://reviews.apache.org/r/20287/diff/


Testing
-------


Thanks,

Alexander Shraer


Re: Review Request 20287: Leader should timeout when loosing quorum of either old or new config

Posted by Alexander Shraer <sh...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20287/
-----------------------------------------------------------

(Updated April 14, 2014, 5:33 a.m.)


Review request for zookeeper.


Bugs: zookeeper-1699
    https://issues.apache.org/jira/browse/zookeeper-1699


Repository: zookeeper


Description
-------

Change the way syncSet is calculated in leader.java, to account for the proposed config, which also should have a quorum.
Leader should time out when loosing quorum of either old or newly proposed config.


Diffs (updated)
-----

  ./src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1586860 
  ./src/java/test/org/apache/zookeeper/test/ReconfigTest.java 1586860 

Diff: https://reviews.apache.org/r/20287/diff/


Testing
-------


Thanks,

Alexander Shraer


Re: Review Request 20287: Leader should timeout when loosing quorum of either old or new config

Posted by Alexander Shraer <sh...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20287/
-----------------------------------------------------------

(Updated April 13, 2014, 6:23 a.m.)


Review request for zookeeper.


Bugs: zookeeper-1699
    https://issues.apache.org/jira/browse/zookeeper-1699


Repository: zookeeper


Description
-------

Change the way syncSet is calculated in leader.java, to account for the proposed config, which also should have a quorum.
Leader should time out when loosing quorum of either old or newly proposed config.


Diffs (updated)
-----

  ./src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1586860 
  ./src/java/test/org/apache/zookeeper/test/ReconfigTest.java 1586860 

Diff: https://reviews.apache.org/r/20287/diff/


Testing
-------


Thanks,

Alexander Shraer


Re: Review Request 20287: Leader should timeout when loosing quorum of either old or new config

Posted by Alexander Shraer <sh...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20287/
-----------------------------------------------------------

(Updated April 13, 2014, 2:03 a.m.)


Review request for zookeeper.


Bugs: zookeeper-1699
    https://issues.apache.org/jira/browse/zookeeper-1699


Repository: zookeeper


Description
-------

Change the way syncSet is calculated in leader.java, to account for the proposed config, which also should have a quorum.
Leader should time out when loosing quorum of either old or newly proposed config.


Diffs (updated)
-----

  ./src/java/main/org/apache/zookeeper/server/quorum/Leader.java 1586860 

Diff: https://reviews.apache.org/r/20287/diff/


Testing
-------


Thanks,

Alexander Shraer