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


##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/RebalanceConfig.java:
##########
@@ -83,6 +83,13 @@ public class RebalanceConfig {
   @ApiModelProperty(example = "false")
   private boolean _bestEfforts = false;
 
+  // Whether to enforce Minimal Data Movement Algorithm (only effective if 
instance assignment config is set, and if
+  // bootstrap is false). If set to false, the minimizeDataMovement flag in 
the table config will be used to determine
+  // whether to run the Minimal Data Movement Algorithm.
+  @JsonProperty("minimizeDataMovement")
+  @ApiModelProperty(example = "TRUE")
+  private String _minimizeDataMovement = "TRUE";

Review Comment:
   Ended up with an Enum method.



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