Gabriella Lotz has posted comments on this change. ( http://gerrit.cloudera.org:8080/25005 )
Change subject: [auto_rebalancer] enforce per-server move cap ...................................................................... Patch Set 5: > Patch Set 4: Code-Review+2 > > LGTM. > > Please make sure the pre-commit test failures are unrelated to this > changelist. I noticed AddTserver in auto_leader_rebalancer-test.cc failed on the debug and release builds. It turned out the test is too slow now that the per-server cap is enforced, so I raised auto_rebalancing_max_moves_per_server to 5 in the test fixture. Without that change AddTserver failed locally, and with it, it passed 20 out of 20 repeated runs in a debug build. -- 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: comment 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]> Gerrit-Comment-Date: Thu, 22 Oct 2026 14:08:49 +0000 Gerrit-HasComments: No
