kangkaisen commented on a change in pull request #5010:
URL: https://github.com/apache/incubator-doris/pull/5010#discussion_r546702636
##########
File path: fe/fe-core/src/main/java/org/apache/doris/common/Config.java
##########
@@ -1010,6 +1010,18 @@
@ConfField(mutable = true, masterOnly = true)
public static int max_balancing_tablets = 100;
+ // Rebalancer type(ignore case): BeLoad, Partition. If type parse failed,
use BeLoad as default.
+ @ConfField(masterOnly = true)
+ public static String rebalancer_type = "BeLoad";
+
+ // Valid only if use PartitionRebalancer.
+ @ConfField(mutable = true, masterOnly = true)
+ public static long rebalance_move_expire_after_access = 600; // 600s
Review comment:
If only valid for PartitionRebalancer, the name would better add a
partition prefix.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
