You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2018/01/09 23:19:50 UTC
kudu git commit: [raft_consensus-itest] fix for Test_KUDU_1735
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 <ad...@cloudera.com>
Tested-by: Alexey Serbin <as...@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 <as...@cloudera.com>
Authored: Tue Jan 9 14:55:18 2018 -0800
Committer: Alexey Serbin <as...@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));