Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/25005 )
Change subject: [auto_rebalancer] enforce per-server move cap ...................................................................... Patch Set 5: Code-Review+2 > > 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. Thanks for checking it out. LGTM. -- 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: Fri, 23 Oct 2026 22:15:20 +0000 Gerrit-HasComments: No
