kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From t...@apache.org
Subject [1/4] kudu git commit: raft_consensus-itest: fix reliance on std::unordered_map iteration order
Date Thu, 15 Sep 2016 06:57:34 GMT
Repository: kudu
Updated Branches:
  refs/heads/master 9a12926f8 -> 1eb24183a


raft_consensus-itest: fix reliance on std::unordered_map iteration order

Fixes another instance similar to cb8f17589486 in which we were relying
on the iteration order of an unordered map. This caused the test to fail
in some environments.

Change-Id: Id6fd1391bfb1be0a67f66f73286e3c81127c5f26
Reviewed-on: http://gerrit.cloudera.org:8080/4422
Reviewed-by: David Ribeiro Alves <dralves@apache.org>
Tested-by: Kudu Jenkins


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

Branch: refs/heads/master
Commit: 590207debbad4bda7165c877be6e1aa636763224
Parents: 9a12926
Author: Todd Lipcon <todd@apache.org>
Authored: Wed Sep 14 15:30:07 2016 -0700
Committer: Todd Lipcon <todd@apache.org>
Committed: Thu Sep 15 03:36:49 2016 +0000

----------------------------------------------------------------------
 src/kudu/integration-tests/raft_consensus-itest.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/590207de/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 22a7ada..1bbef3c 100644
--- a/src/kudu/integration-tests/raft_consensus-itest.cc
+++ b/src/kudu/integration-tests/raft_consensus-itest.cc
@@ -2568,7 +2568,7 @@ TEST_F(RaftConsensusITest, TestUpdateConsensusErrorNonePrepared) {
 
   // Configure the first server to fail all on prepare.
   TServerDetails *replica_ts = tservers[0];
-  ASSERT_OK(cluster_->SetFlag(cluster_->tablet_server(0),
+  ASSERT_OK(cluster_->SetFlag(cluster_->tablet_server_by_uuid(replica_ts->uuid()),
                 "follower_fail_all_prepare", "true"));
 
   // Pretend to be the leader and send a request that should return an error.


Mime
View raw message