Github user hanm commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/588#discussion_r210481082
--- Diff: src/java/main/org/apache/zookeeper/server/quorum/Leader.java ---
@@ -1165,6 +1165,58 @@ synchronized public long
startForwarding(LearnerHandler handler,
}
// VisibleForTesting
protected final Set<Long> connectingFollowers = new HashSet<Long>();
+
+ private volatile boolean quitWaitForEpoch = false;
+ private volatile long timeStartWaitForEpoch = -1;
+ private volatile SyncedLearnerTracker voteSet;
+
+ public static final String MIN_TIME_WAIT_FOR_EPOCH =
"zookeeper.leader.minTimeToWaitForEpoch";
--- End diff --
MAX_TIME_TO_WAIT_FOR_EPOCH feels more intuitive for me. Also there should
be some documentation of this new property in admin doc so users know its
existence and how to configure it.
---
