[master] fix misprint in VerifyMastersGetHostPorts()
There was a misprint in the VerifyMastersGetHostPorts() function
used in multi-master startup -- 'return Status::OK()' line was missing
in the 'if()' clause when the masters don't agree on the current Raft
config's index.
Kotomi from #getkudu community Slack found and reported this issue.
Thank you for the contribution!
Change-Id: I8e2d623ac09bb0eb8e25e1a38cfba6b5b9819c78
Reviewed-on: http://gerrit.cloudera.org:8080/17844
Tested-by: Kudu Jenkins
Reviewed-by: Andrew Wong <awong@cloudera.com>
diff --git a/src/kudu/master/master_runner.cc b/src/kudu/master/master_runner.cc
index f4c194a..fffd28f 100644
--- a/src/kudu/master/master_runner.cc
+++ b/src/kudu/master/master_runner.cc
@@ -249,6 +249,7 @@
LOG(INFO) << Substitute("Existing masters have differing Raft config indexes: $0 vs $1",
committed_config_index, cstate.committed_config().opid_index());
*needs_retry = true;
+ return Status::OK();
}
const auto& config = cstate.committed_config();
set<string> uuids;
@@ -256,7 +257,6 @@
EmplaceIfNotPresent(&uuids, p.permanent_uuid());
}
each_remote_masters_master_uuids.emplace_back(std::move(uuids));
-
}
if (!leader_hp->Initialized()) {
LOG(INFO) << Substitute("No leader master found from master $0", local_uuid);
