From 86b87f7b32dde5da1b27b538b810004b0c0ab293 Mon Sep 17 00:00:00 2001 From: "Xiaotian (Jackie) Jiang" Date: Tue, 6 Dec 2022 15:33:42 -0800 Subject: [PATCH] Add back the Helix workaround for missing IS change --- .../org/apache/pinot/controller/helix/ControllerTest.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java index 0ad886a769c4..3259c5148fa3 100644 --- a/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java +++ b/pinot-controller/src/test/java/org/apache/pinot/controller/helix/ControllerTest.java @@ -41,6 +41,7 @@ import org.apache.helix.HelixManagerFactory; import org.apache.helix.InstanceType; import org.apache.helix.NotificationContext; +import org.apache.helix.model.ClusterConfig; import org.apache.helix.model.HelixConfigScope; import org.apache.helix.model.Message; import org.apache.helix.model.ResourceConfig; @@ -253,6 +254,10 @@ public void startController(Map properties) case PINOT_ONLY: _helixAdmin = _helixResourceManager.getHelixAdmin(); _propertyStore = _helixResourceManager.getPropertyStore(); + // TODO: Enable periodic rebalance per 10 seconds as a temporary work-around for the Helix issue: + // https://github.com/apache/helix/issues/331 and https://github.com/apache/helix/issues/2309. + // Remove this after Helix fixing the issue. + configAccessor.set(scope, ClusterConfig.ClusterConfigProperty.REBALANCE_TIMER_PERIOD.name(), "10000"); break; case HELIX_ONLY: _helixAdmin = _helixManager.getClusterManagmentTool();