You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by ca...@apache.org on 2022/10/13 13:47:54 UTC

[iotdb] 01/01: remove comment code

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

caogaofei pushed a commit to branch beyyes/remove_confignode_improvement
in repository https://gitbox.apache.org/repos/asf/iotdb.git

commit f2f33f684c82eb7d1cf8a0021aeb0ce91377b00b
Author: Beyyes <cg...@foxmail.com>
AuthorDate: Thu Oct 13 21:46:45 2022 +0800

    remove comment code
---
 .../service/thrift/ConfigNodeRPCServiceProcessor.java     | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/confignode/src/main/java/org/apache/iotdb/confignode/service/thrift/ConfigNodeRPCServiceProcessor.java b/confignode/src/main/java/org/apache/iotdb/confignode/service/thrift/ConfigNodeRPCServiceProcessor.java
index 33303f057a..e8462add61 100644
--- a/confignode/src/main/java/org/apache/iotdb/confignode/service/thrift/ConfigNodeRPCServiceProcessor.java
+++ b/confignode/src/main/java/org/apache/iotdb/confignode/service/thrift/ConfigNodeRPCServiceProcessor.java
@@ -458,21 +458,6 @@ public class ConfigNodeRPCServiceProcessor implements IConfigNodeRPCService.Ifac
               "remove ConsensusGroup failed because internal failure. See other logs for more details");
     }
 
-    //    List<TConfigNodeLocation> registeredConfigNodes =
-    //            configManager.getNodeManager().getRegisteredConfigNodes();
-    //    registeredConfigNodes.remove(configNodeLocation);
-    //    try {
-    //      // The removed ConfigNode also need to update the confignode-system.properties file
-    //      // Because user may execute remove-confignode.sh in this node
-    //      SystemPropertiesUtils.storeConfigNodeList(new ArrayList<>(registeredConfigNodes));
-    //      LOGGER.info(
-    //              "Updated ConfigNode information files after deleteConfigNodePeer. {}.",
-    //              registeredConfigNodes);
-    //    } catch (IOException e) {
-    //      LOGGER.error("Current ConfigNode to be removed can't update ConfigNode information.",
-    // e);
-    //    }
-
     return new TSStatus(TSStatusCode.SUCCESS_STATUS.getStatusCode())
         .setMessage("remove ConsensusGroup success.");
   }