J-HowHuang commented on code in PR #15110:
URL: https://github.com/apache/pinot/pull/15110#discussion_r1985895623


##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -612,9 +612,9 @@ public RebalanceResult rebalance(
       @QueryParam("reassignInstances") boolean reassignInstances,
       @ApiParam(value = "Whether to reassign CONSUMING segments for real-time 
table") @DefaultValue("false")
       @QueryParam("includeConsuming") boolean includeConsuming,
-      @ApiParam(value = "TRUE|FALSE|DEFAULT, whether to enable minimize data 
movement on rebalance, DEFAULT will use "
-          + "the minimizeDataMovement in table config") @DefaultValue("TRUE")
-      @QueryParam("minimizeDataMovement") String minimizeDataMovement,
+      @ApiParam(value = "Whether to enable minimize data movement on 
rebalance, DEFAULT will use "
+          + "the minimizeDataMovement in table config") @DefaultValue("ENABLE")
+      @QueryParam("minimizeDataMovement") 
RebalanceConfig.MinimizeDataMovementOptions minimizeDataMovement,

Review Comment:
   it returns HTTP 404
   



-- 
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