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 2017/12/14 19:02:11 UTC
kudu git commit: [itests] fix build
Repository: kudu
Updated Branches:
refs/heads/master bb2620299 -> dbc84a2d7
[itests] fix build
This is a follow-up for 7c8fc625e72ec2bf6ef614df2ec25346e1a0f7b7, which
became a bit outdated after bb2620299c45b3db90590236298a497a619c0bda was
merged.
This patch does not contain any functional changes.
Change-Id: I466d941f8634e37b68dd85f29d39f1eb02b26173
Reviewed-on: http://gerrit.cloudera.org:8080/8844
Reviewed-by: Mike Percy <mp...@apache.org>
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/dbc84a2d
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/dbc84a2d
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/dbc84a2d
Branch: refs/heads/master
Commit: dbc84a2d7469e2de97fe9e35f5b5443e94f2a790
Parents: bb26202
Author: Alexey Serbin <as...@cloudera.com>
Authored: Thu Dec 14 10:55:14 2017 -0800
Committer: Alexey Serbin <as...@cloudera.com>
Committed: Thu Dec 14 19:01:37 2017 +0000
----------------------------------------------------------------------
.../integration-tests/raft_consensus_nonvoter-itest.cc | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/dbc84a2d/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
----------------------------------------------------------------------
diff --git a/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc b/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
index 11fbbbe..022fa4f 100644
--- a/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
+++ b/src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc
@@ -1585,14 +1585,13 @@ TEST_F(RaftConsensusNonVoterITest, RestartClusterWithNonVoter) {
}
workload.StopAndJoin();
- vector<ExternalTabletServer*> servers_without_replica;
- NO_FATALS(GetServersWithoutReplica(&servers_without_replica));
- ASSERT_EQ(4, servers_without_replica.size());
-
- ExternalTabletServer* ts_with_replica = GetServerWithReplica();
+ const auto server_uuids_with_replica = GetServersWithReplica(tablet_id_);
+ ASSERT_FALSE(server_uuids_with_replica.empty());
+ const string& failed_replica_uuid = server_uuids_with_replica.front();
+ ExternalTabletServer* ts_with_replica =
+ cluster_->tablet_server_by_uuid(failed_replica_uuid);
ASSERT_NE(nullptr, ts_with_replica);
ts_with_replica->Shutdown();
- const string& failed_replica_uuid = ts_with_replica->uuid();
bool has_leader = false;
TabletLocationsPB tablet_locations;