You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ratis.apache.org by sz...@apache.org on 2020/10/21 14:37:57 UTC
[incubator-ratis] branch master updated: RATIS-1101. Fix Failed UT:
RaftBasicTests.runTestOldLeaderNotCommit:IndexOutOfBounds (#229).
Contributed by Rui Wang
This is an automated email from the ASF dual-hosted git repository.
szetszwo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-ratis.git
The following commit(s) were added to refs/heads/master by this push:
new a80fc6c RATIS-1101. Fix Failed UT: RaftBasicTests.runTestOldLeaderNotCommit:IndexOutOfBounds (#229). Contributed by Rui Wang
a80fc6c is described below
commit a80fc6cf47eed1aaa977577c8e12ed23d5c949a2
Author: Rui Wang <am...@users.noreply.github.com>
AuthorDate: Wed Oct 21 07:37:46 2020 -0700
RATIS-1101. Fix Failed UT: RaftBasicTests.runTestOldLeaderNotCommit:IndexOutOfBounds (#229). Contributed by Rui Wang
---
.../src/test/java/org/apache/ratis/RaftBasicTests.java | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/ratis-server/src/test/java/org/apache/ratis/RaftBasicTests.java b/ratis-server/src/test/java/org/apache/ratis/RaftBasicTests.java
index 48d310b..42fd159 100644
--- a/ratis-server/src/test/java/org/apache/ratis/RaftBasicTests.java
+++ b/ratis-server/src/test/java/org/apache/ratis/RaftBasicTests.java
@@ -228,9 +228,14 @@ public abstract class RaftBasicTests<CLUSTER extends MiniRaftCluster>
List<RaftServerImpl> followers = cluster.getFollowers();
final RaftServerImpl followerToCommit = followers.get(0);
- for (int i = 1; i < NUM_SERVERS - 1; i++) {
- RaftServerImpl follower = followers.get(i);
- cluster.killServer(follower.getId());
+ try {
+ for (int i = 1; i < NUM_SERVERS - 1; i++) {
+ RaftServerImpl follower = followers.get(i);
+ cluster.killServer(follower.getId());
+ }
+ } catch (IndexOutOfBoundsException e) {
+ throw new org.junit.AssumptionViolatedException("The assumption is follower.size() = NUM_SERVERS - 1, "
+ + "actual NUM_SERVERS is " + NUM_SERVERS + ", and actual follower.size() is " + followers.size(), e);
}
SimpleMessage[] messages = SimpleMessage.create(1);