neuyilan commented on a change in pull request #2740:
URL: https://github.com/apache/iotdb/pull/2740#discussion_r585306792
##########
File path: cluster/src/main/java/org/apache/iotdb/cluster/ClusterMain.java
##########
@@ -179,10 +180,11 @@ private static void startServerCheck() throws
StartupException {
// assert this node is in seed nodes list
Node localNode = new Node();
localNode
- .setIp(config.getClusterRpcIp())
+ .setIp(config.getInternalIp())
.setMetaPort(config.getInternalMetaPort())
.setDataPort(config.getInternalDataPort())
- .setClientPort(config.getClusterRpcPort());
+ .setClientPort(config.getClusterRpcPort())
Review comment:
We should split the cluster_rpc_port and the rpc_port due to the
following reasons, and because the cluster_rpc_port have been
Initializatiioned, so it's actually cluster_rpc_port
the rpc_port of a single module can not be used directly, because when the
debug mode(open_server_rpc_port=true) is turned on, the rpc_port of a single
module may be different from that of a distributed cluster.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
