kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: [itests] update on raft_consensus_stress-itest
Date Sat, 10 Mar 2018 01:08:39 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 92486c83d -> 29dbc7e41


[itests] update on raft_consensus_stress-itest

Updates on raft_consensus_stress-itest after running it in 3-2-3 mode.

I found that sometimes 30 iterations of ksck was not enough to converge
if running at dist-test machines with the 3-2-3 re-replication mode,
but 50 iterations plus delay of one heartbeat interval between retries
was enough to make it more stable for the 3-2-3 mode runs.

Change-Id: I6762f1783f4e926225befe3e49f4e12b58a27dd1
Reviewed-on: http://gerrit.cloudera.org:8080/9567
Reviewed-by: Adar Dembo <adar@cloudera.com>
Tested-by: Alexey Serbin <aserbin@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/29dbc7e4
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/29dbc7e4
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/29dbc7e4

Branch: refs/heads/master
Commit: 29dbc7e411bbd1d0e542185880163642703ca5ba
Parents: 92486c8
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Fri Mar 9 15:35:18 2018 -0800
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Sat Mar 10 01:05:46 2018 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/raft_consensus_stress-itest.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/29dbc7e4/src/kudu/integration-tests/raft_consensus_stress-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/raft_consensus_stress-itest.cc b/src/kudu/integration-tests/raft_consensus_stress-itest.cc
index 7857930..1531caa 100644
--- a/src/kudu/integration-tests/raft_consensus_stress-itest.cc
+++ b/src/kudu/integration-tests/raft_consensus_stress-itest.cc
@@ -65,7 +65,7 @@ DEFINE_int32(test_follower_unavailable_considered_failed_sec,
 DEFINE_int32(test_heartbeat_interval_ms,
              250 * kBuildCfgFactor,
              "Interval at which the TS heartbeats to the master.");
-DEFINE_int32(test_max_ksck_failures, 30,
+DEFINE_int32(test_max_ksck_failures, 50,
              "Maximum number of ksck failures in a row to tolerate before "
              "considering the test as failed.");
 // GLOG_FATAL:    3
@@ -257,6 +257,7 @@ TEST_F(RaftConsensusStressITest, RemoveReplaceInCycle) {
       if (ksck_failures_in_a_row > FLAGS_test_max_ksck_failures) {
         break;
       }
+      SleepFor(MonoDelta::FromMilliseconds(FLAGS_test_raft_heartbeat_interval_ms));
       continue;
     }
     ksck_failures_in_a_row = 0;


Mime
View raw message