Hello Marton Greber, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24285
to look at the new patch set (#8).
Change subject: KUDU-3732 Get consensus state from CatalogManager in
auto-rebalancer
......................................................................
KUDU-3732 Get consensus state from CatalogManager in auto-rebalancer
AutoRebalancerTask::CheckMoveCompleted() previously issued a
GetConsensusState RPC to the tablet leader on every polling cycle
to check whether a replica move had completed. Replace this RPC with
a read from CatalogManager, which is populated via tserver-to-master
heartbeats.
ExecuteMoves now records the committed config opid_index from
CatalogManager before sending BulkChangeConfig, storing it in
ReplicaMove.config_opid_idx. CheckMoveCompleted uses this as a
freshness gate: if the opid_index in CatalogManager has not advanced
past the stored value, the heartbeat carrying the new config has not
yet arrived and the check waits rather than applying completion or
error logic.
Add CatalogManager::GetTabletConsensusState() to expose the
committed consensus state from the in-memory tablet map.
Change-Id: Iba021539fbfaca5905fbbf4abbb6e2a61db1f7dc
---
M src/kudu/master/auto_rebalancer-test.cc
M src/kudu/master/auto_rebalancer.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
4 files changed, 311 insertions(+), 52 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/24285/8
--
To view, visit http://gerrit.cloudera.org:8080/24285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iba021539fbfaca5905fbbf4abbb6e2a61db1f7dc
Gerrit-Change-Number: 24285
Gerrit-PatchSet: 8
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
