This is an automated email from the ASF dual-hosted git repository.

rongr pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 173916e42f Add back the Helix workaround for missing IS change (#9921)
173916e42f is described below

commit 173916e42faee24dc1bb3f3d4de2c30bf2efd7a5
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Tue Dec 6 20:19:09 2022 -0800

    Add back the Helix workaround for missing IS change (#9921)
---
 .../test/java/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 0ad886a769..3259c5148f 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.HelixManager;
 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 class ControllerTest {
       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();


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to