somandal commented on code in PR #15617:
URL: https://github.com/apache/pinot/pull/15617#discussion_r2080689596


##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -625,6 +625,13 @@ public RebalanceResult rebalance(
           + "more servers.") @DefaultValue("false") @QueryParam("lowDiskMode") 
boolean lowDiskMode,
       @ApiParam(value = "Whether to use best-efforts to rebalance (not fail 
the rebalance when the no-downtime "
           + "contract cannot be achieved)") @DefaultValue("false") 
@QueryParam("bestEfforts") boolean bestEfforts,
+      @ApiParam(value = "How many maximum segment adds per server to update in 
the IdealState in each step. For "
+          + "non-strict replica group based assignment, this number will be 
the closest possible without splitting up "
+          + "a single segment's step's replicas across steps (so some servers 
may get fewer segments). For strict "
+          + "replica group based assignment, this is a per-server best effort 
value since each partition of a replica "
+          + "group must be moved as a whole and at least one partition in a 
replica group should be moved. A value of "
+          + "-1 is used to disable batching (unlimited segments).")

Review Comment:
   done



##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -625,6 +625,13 @@ public RebalanceResult rebalance(
           + "more servers.") @DefaultValue("false") @QueryParam("lowDiskMode") 
boolean lowDiskMode,
       @ApiParam(value = "Whether to use best-efforts to rebalance (not fail 
the rebalance when the no-downtime "
           + "contract cannot be achieved)") @DefaultValue("false") 
@QueryParam("bestEfforts") boolean bestEfforts,
+      @ApiParam(value = "How many maximum segment adds per server to update in 
the IdealState in each step. For "
+          + "non-strict replica group based assignment, this number will be 
the closest possible without splitting up "
+          + "a single segment's step's replicas across steps (so some servers 
may get fewer segments). For strict "

Review Comment:
   done



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to