Gabriella Lotz has posted comments on this change. ( http://gerrit.cloudera.org:8080/23755 )
Change subject: auto_rebalancer: enforce max_moves_per_server per tserver ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/23755/4/src/kudu/master/auto_rebalancer.cc File src/kudu/master/auto_rebalancer.cc: http://gerrit.cloudera.org:8080/#/c/23755/4/src/kudu/master/auto_rebalancer.cc@270 PS4, Line 270: #if DCHECK_IS_ON() : for (const auto& entry : moves_per_tserver_) { : DCHECK_EQ(0, entry.second) << "Tserver " << entry.first << " still has " << entry.second : > nit: to avoid having empty 'for' cycle (and possible compiler warnings), co Done -- To view, visit http://gerrit.cloudera.org:8080/23755 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic1b73045719dc69cc9e3353d318b7381f144a20d Gerrit-Change-Number: 23755 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-Comment-Date: Mon, 19 Jan 2026 10:27:00 +0000 Gerrit-HasComments: Yes
