huangzhaobo99 commented on code in PR #7120:
URL: https://github.com/apache/hadoop/pull/7120#discussion_r1817984036
##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/balancer/Balancer.java:
##########
@@ -1071,6 +1096,12 @@ static BalancerParameters parse(String[] args) {
b.setSortTopNodes(true);
LOG.info("Balancer will sort nodes by" +
" capacity usage percentage to prioritize top used nodes");
+ } else if ("-limitOverUtilizedNum".equalsIgnoreCase(args[i])) {
+ Preconditions.checkArgument(++i < args.length,
+ "limitOverUtilizedNum value is missing: args = " +
Arrays.toString(args));
+ int limitNum = Integer.parseInt(args[i]);
Review Comment:
Now, restricting the list that is greater than the threshold, and in the
future, we will also need to restrict the list that is greater than the average
(aboveAvgUtilized), 0 is legal.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
