You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by lavacat <gi...@git.apache.org> on 2018/04/20 00:59:30 UTC
[GitHub] zookeeper pull request #503: ZOOKEEPER-2959: ignore accepted epoch and LEADE...
GitHub user lavacat opened a pull request:
https://github.com/apache/zookeeper/pull/503
ZOOKEEPER-2959: ignore accepted epoch and LEADERINFO ack from observers
https://issues.apache.org/jira/browse/ZOOKEEPER-2959
- added getVotingMembers check for id in getEpochToPropose and waitForEpochAck
- removed unused learnerType param in waitForNewLeaderAck
- unit tests
- refactored common test helpers into ZabUtils
credit: Xiang Yongqiang (https://github.com/xyq000) for original PR and reporting the issue
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/lavacat/zookeeper branch-3.5
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zookeeper/pull/503.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #503
----
commit d7181d65f66adcfc0fecda2670580e2d2b8ddccb
Author: Bogdan Kanivets <bk...@...>
Date: 2018-04-20T00:02:59Z
ZOOKEEPER-2959: ignore accepted epoch and LEADERINFO ack from observers
https://issues.apache.org/jira/browse/ZOOKEEPER-2959
- added getVotingMembers check for id in getEpochToPropose and waitForEpochAck
- removed unused learnerType param in waitForNewLeaderAck
- unit tests
- refactored common test helpers into ZabUtils
credit: Xiang Yongqiang (https://github.com/xyq000) for original PR and reporting the issue
----
---
[GitHub] zookeeper issue #503: ZOOKEEPER-2959: ignore accepted epoch and LEADERINFO a...
Posted by shralex <gi...@git.apache.org>.
Github user shralex commented on the issue:
https://github.com/apache/zookeeper/pull/503
+1 looks good
---
[GitHub] zookeeper pull request #503: ZOOKEEPER-2959: ignore accepted epoch and LEADE...
Posted by lavacat <gi...@git.apache.org>.
Github user lavacat closed the pull request at:
https://github.com/apache/zookeeper/pull/503
---