Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23909 )

Change subject: KUDU-3728 Add range-aware rebalancing to auto_rebalancer
......................................................................


Patch Set 6: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23909/6/src/kudu/master/auto_rebalancer.cc
File src/kudu/master/auto_rebalancer.cc:

http://gerrit.cloudera.org:8080/#/c/23909/6/src/kudu/master/auto_rebalancer.cc@617
PS6, Line 617: static_cast<uint16_t>(static_cast<uint8_t>(range_key_begin[i])
There was a discussion thread on this in PS1.

Continuing it here as of PS6 (at least to reduce risk of missed/non-addresses 
comments), do you want to have a discrepancy between the code in ksck and here 
once this changelist is merged?



--
To view, visit http://gerrit.cloudera.org:8080/23909
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idf716bef8c21395da98dd7c9e33001190d9b5587
Gerrit-Change-Number: 23909
Gerrit-PatchSet: 6
Gerrit-Owner: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Gabriella Lotz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 03 Mar 2026 18:29:20 +0000
Gerrit-HasComments: Yes

Reply via email to