Skip to content

Commit

Permalink
[controller] Set rebalancing strategy to Waged instead of Crush when …
Browse files Browse the repository at this point in the history
…adding a resource to controller cluster (#1197)
  • Loading branch information
kvargha authored Sep 24, 2024
1 parent 5d92c7d commit 1df15f8
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
import org.apache.helix.cloud.constants.CloudProvider;
import org.apache.helix.controller.rebalancer.DelayedAutoRebalancer;
import org.apache.helix.controller.rebalancer.strategy.AutoRebalanceStrategy;
import org.apache.helix.controller.rebalancer.strategy.CrushRebalanceStrategy;
import org.apache.helix.controller.rebalancer.waged.WagedRebalancer;
import org.apache.helix.manager.zk.ZKHelixAdmin;
import org.apache.helix.manager.zk.ZKHelixManager;
import org.apache.helix.manager.zk.ZNRecordSerializer;
Expand Down Expand Up @@ -167,8 +167,7 @@ public void addVeniceStorageClusterToControllerCluster(String clusterName) {
VeniceControllerClusterConfig config = multiClusterConfigs.getControllerConfig(clusterName);
IdealState idealState = helixAdmin.getResourceIdealState(controllerClusterName, clusterName);
idealState.setMinActiveReplicas(controllerClusterReplicaCount);
idealState.setRebalancerClassName(DelayedAutoRebalancer.class.getName());
idealState.setRebalanceStrategy(CrushRebalanceStrategy.class.getName());
idealState.setRebalancerClassName(WagedRebalancer.class.getName());

String instanceGroupTag = config.getControllerResourceInstanceGroupTag();
if (!instanceGroupTag.isEmpty()) {
Expand Down

0 comments on commit 1df15f8

Please sign in to comment.