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

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


The following commit(s) were added to refs/heads/master by this push:
     new f3b6c08  Remove commented out code in participant manager (#1029)
f3b6c08 is described below

commit f3b6c0809793452b97b0e403f7a2df08acb7de36
Author: Meng Zhang <[email protected]>
AuthorDate: Tue May 26 14:50:02 2020 -0700

    Remove commented out code in participant manager (#1029)
---
 .../java/org/apache/helix/manager/zk/ParticipantManager.java | 12 ------------
 1 file changed, 12 deletions(-)

diff --git 
a/helix-core/src/main/java/org/apache/helix/manager/zk/ParticipantManager.java 
b/helix-core/src/main/java/org/apache/helix/manager/zk/ParticipantManager.java
index 6e9d579..704f247 100644
--- 
a/helix-core/src/main/java/org/apache/helix/manager/zk/ParticipantManager.java
+++ 
b/helix-core/src/main/java/org/apache/helix/manager/zk/ParticipantManager.java
@@ -201,18 +201,6 @@ public class ParticipantManager {
           String domain = cloudInstanceInformation
               
.get(CloudInstanceInformation.CloudInstanceField.FAULT_DOMAIN.name()) + 
_instanceName;
 
-          // Disable the verification for now
-          /*String cloudIdInRemote = cloudInstanceInformation
-              
.get(CloudInstanceInformation.CloudInstanceField.INSTANCE_SET_NAME.name());
-          String cloudIdInConfig = 
_configAccessor.getCloudConfig(_clusterName).getCloudID();
-
-          // validate that the instance is auto registering to the correct 
cluster
-          if (!cloudIdInRemote.equals(cloudIdInConfig)) {
-            throw new IllegalArgumentException(String.format(
-                "cloudId in config: %s is not consistent with cloudId from 
remote: %s. The instance is auto registering to a wrong cluster.",
-                cloudIdInConfig, cloudIdInRemote));
-          }*/
-
           InstanceConfig instanceConfig = 
HelixUtil.composeInstanceConfig(_instanceName);
           instanceConfig.setDomain(domain);
           _helixAdmin.addInstance(_clusterName, instanceConfig);

Reply via email to