Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24285 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/24285 Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Marton Greber <[email protected]> Tested-by: Marton Greber <[email protected]> --- 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 M src/kudu/master/ts_descriptor.h 5 files changed, 349 insertions(+), 52 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Marton Greber: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Iba021539fbfaca5905fbbf4abbb6e2a61db1f7dc Gerrit-Change-Number: 24285 Gerrit-PatchSet: 11 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]>
