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