This is an automated email from the ASF dual-hosted git repository.
jackie 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 bcc9587eda Remove the temporary workaround for a Helix bug (#9634)
bcc9587eda is described below
commit bcc9587eda0581926d8ae9d6bd179474722e630d
Author: Xiaotian (Jackie) Jiang <[email protected]>
AuthorDate: Thu Oct 20 17:45:24 2022 -0700
Remove the temporary workaround for a Helix bug (#9634)
---
.../test/java/org/apache/pinot/controller/helix/ControllerTest.java | 5 -----
1 file changed, 5 deletions(-)
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 3a5eea42ba..64837223a9 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,7 +41,6 @@ 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;
@@ -257,10 +256,6 @@ 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. 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]
