kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ale...@apache.org
Subject kudu git commit: [raft_consensus-itest] fix for Test_KUDU_1735
Date Tue, 09 Jan 2018 23:19:50 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 746d0a5cd -> 8bcc80eec


[raft_consensus-itest] fix for Test_KUDU_1735

Quick fir for Test_KUDU_1735 after enabling 3-4-3 replication mode
by default.  For some reason, this test slipped under radar and
I discovered that it fails only now (the other relevant tests were
already updated).

I tried to make this test work for 3-4-3, but it was not successful
yet.  So, to fix the build, let's push this easy fix first.

This is a follow-up for 1ac10d5364db507e5052f5dc13c19d32f62dc4f6.

Change-Id: I8d96b66d528f90f7611ef32726401e3208b0e47d
Reviewed-on: http://gerrit.cloudera.org:8080/8987
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/8bcc80ee
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/8bcc80ee
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/8bcc80ee

Branch: refs/heads/master
Commit: 8bcc80eec075321fa4444ef26b2a0ccac12ac9d7
Parents: 746d0a5
Author: Alexey Serbin <aserbin@cloudera.com>
Authored: Tue Jan 9 14:55:18 2018 -0800
Committer: Alexey Serbin <aserbin@cloudera.com>
Committed: Tue Jan 9 23:19:05 2018 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/raft_consensus-itest.cc | 6 ++++++
 1 file changed, 6 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/8bcc80ee/src/kudu/integration-tests/raft_consensus-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/raft_consensus-itest.cc b/src/kudu/integration-tests/raft_consensus-itest.cc
index fdda2da..281ef57 100644
--- a/src/kudu/integration-tests/raft_consensus-itest.cc
+++ b/src/kudu/integration-tests/raft_consensus-itest.cc
@@ -2402,10 +2402,16 @@ TEST_F(RaftConsensusITest, TestChangeConfigRejectedUnlessNoopReplicated)
{
 TEST_F(RaftConsensusITest, Test_KUDU_1735) {
   const MonoDelta kTimeout = MonoDelta::FromSeconds(10);
   const vector<string> kTsFlags = {
+    // This scenario uses 'manual election' mode and assumes no leader change.
     "--enable_leader_failure_detection=false",
+    // This test is specific for the 3-2-3 replica management scheme.
+    "--raft_prepare_replacement_before_eviction=false",
   };
   const vector<string> kMasterFlags = {
+    // This scenario uses 'manual election' mode and assumes no leader change.
     "--catalog_manager_wait_for_new_tablets_to_elect_leader=false",
+    // This test is specific for the 3-2-3 replica management scheme.
+    "--raft_prepare_replacement_before_eviction=false",
   };
 
   NO_FATALS(BuildAndStart(kTsFlags, kMasterFlags));


Mime
View raw message