Hello Marton Greber, Zoltan Chovan, Alexey Serbin, Kudu Jenkins, Abhishek
Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/25005
to look at the new patch set (#5).
Change subject: [auto_rebalancer] enforce per-server move cap
......................................................................
[auto_rebalancer] enforce per-server move cap
The auto-rebalancer never enforced its per-server move cap. It budgeted
a whole batch at max_moves_per_server * num_tservers and checked
moves_per_tserver_, which is empty at selection time, so one batch could
pile many moves onto a single server (the most loaded one, which the cap
exists to protect).
Track per-server move counts within the batch and skip a move once
either endpoint reaches the cap.
Add MaxMovesPerServerEnforcedWithinOneBatch, which is red before this
change: a 6-move batch puts 3 moves on one server with the cap at 1.
Change-Id: I9f6b9b200c24d215435329a37e953d75fe923c02
Co-authored-by: Cursor <[email protected]>
---
M src/kudu/master/auto_leader_rebalancer-test.cc
M src/kudu/master/auto_rebalancer-test.cc
M src/kudu/master/auto_rebalancer.cc
3 files changed, 123 insertions(+), 14 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/25005/5
--
To view, visit http://gerrit.cloudera.org:8080/25005
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9f6b9b200c24d215435329a37e953d75fe923c02
Gerrit-Change-Number: 25005
Gerrit-PatchSet: 5
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[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]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
