This is an automated email from the ASF dual-hosted git repository.
jackietien pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iotdb.git
The following commit(s) were added to refs/heads/master by this push:
new d95b453b4f [IOTDB-3687] Rename config_nodes to target_config_nodes
(#6495)
d95b453b4f is described below
commit d95b453b4f4f50c79ff348302caf0c27a5ee12f8
Author: YongzaoDan <[email protected]>
AuthorDate: Wed Jun 29 08:34:30 2022 +0800
[IOTDB-3687] Rename config_nodes to target_config_nodes (#6495)
---
.../src/assembly/resources/conf/iotdb-confignode.properties | 8 ++++----
.../org/apache/iotdb/confignode/conf/ConfigNodeDescriptor.java | 7 ++++---
.../org/apache/iotdb/confignode/conf/ConfigNodeStartupCheck.java | 6 +++---
.../test/resources/confignode1conf/iotdb-confignode.properties | 2 +-
.../test/resources/confignode2conf/iotdb-confignode.properties | 2 +-
.../test/resources/confignode3conf/iotdb-confignode.properties | 2 +-
docs/UserGuide/Cluster/Cluster-Setup.md | 4 ++--
docs/UserGuide/Reference/ConfigNode-Config-Manual.md | 4 ++--
docs/UserGuide/Reference/DataNode-Config-Manual.md | 4 ++--
docs/zh/UserGuide/Cluster/Cluster-Setup.md | 4 ++--
docs/zh/UserGuide/Reference/ConfigNode-Config-Manual.md | 4 ++--
docs/zh/UserGuide/Reference/DataNode-Config-Manual.md | 4 ++--
.../src/main/java/org/apache/iotdb/it/env/ConfigNodeWrapper.java | 2 +-
.../src/main/java/org/apache/iotdb/it/env/DataNodeWrapper.java | 2 +-
server/src/assembly/resources/conf/iotdb-datanode.properties | 4 ++--
.../src/main/java/org/apache/iotdb/db/conf/IoTDBDescriptor.java | 2 +-
server/src/test/resources/datanode1conf/iotdb-datanode.properties | 2 +-
server/src/test/resources/datanode2conf/iotdb-datanode.properties | 2 +-
server/src/test/resources/datanode3conf/iotdb-datanode.properties | 2 +-
19 files changed, 34 insertions(+), 33 deletions(-)
diff --git a/confignode/src/assembly/resources/conf/iotdb-confignode.properties
b/confignode/src/assembly/resources/conf/iotdb-confignode.properties
index 2dc311e6da..651caad821 100644
--- a/confignode/src/assembly/resources/conf/iotdb-confignode.properties
+++ b/confignode/src/assembly/resources/conf/iotdb-confignode.properties
@@ -37,14 +37,14 @@ rpc_port=22277
consensus_port=22278
-# Used for connecting to the ConfigNodeGroup
+# At least one running ConfigNode should be set for joining the cluster
# Format: ip:port
# where the ip should be consistent with the target ConfigNode's
confignode_rpc_address,
# and the port should be consistent with the target ConfigNode's
confignode_rpc_port.
-# For the first ConfigNode to start, config_nodes points to its own ip:port.
-# For other ConfigNodes that are started or restarted, config_nodes points to
any running ConfigNode's ip:port.
+# For the first ConfigNode to start, target_config_nodes points to its own
ip:port.
+# For other ConfigNodes that are started or restarted, target_config_nodes
points to any running ConfigNode's ip:port.
# Datatype: String
-config_nodes=0.0.0.0:22277
+target_config_nodes=0.0.0.0:22277
####################
diff --git
a/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeDescriptor.java
b/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeDescriptor.java
index dd887c39da..7b9596ca4c 100644
---
a/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeDescriptor.java
+++
b/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeDescriptor.java
@@ -115,9 +115,10 @@ public class ConfigNodeDescriptor {
Integer.parseInt(
properties.getProperty("consensus_port",
String.valueOf(conf.getConsensusPort()))));
- String targetConfigNode = properties.getProperty("config_nodes", null);
- if (targetConfigNode != null) {
-
conf.setTargetConfigNode(NodeUrlUtils.parseTEndPointUrl(targetConfigNode));
+ // TODO: Enable multiple target_config_nodes
+ String targetConfigNodes = properties.getProperty("target_config_nodes",
null);
+ if (targetConfigNodes != null) {
+
conf.setTargetConfigNode(NodeUrlUtils.parseTEndPointUrl(targetConfigNodes));
}
conf.setSeriesPartitionSlotNum(
diff --git
a/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeStartupCheck.java
b/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeStartupCheck.java
index 7a826d20dc..68bd3754ea 100644
---
a/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeStartupCheck.java
+++
b/confignode/src/main/java/org/apache/iotdb/confignode/conf/ConfigNodeStartupCheck.java
@@ -83,12 +83,12 @@ public class ConfigNodeStartupCheck {
/** Check whether the global configuration of the cluster is correct */
private void checkGlobalConfig() throws ConfigurationException {
// When the ConfigNode consensus protocol is set to StandAlone,
- // the config_nodes needs to point to itself
+ // the target_config_nodes needs to point to itself
if
(conf.getConfigNodeConsensusProtocolClass().equals(ConsensusFactory.StandAloneConsensus)
&& (!conf.getRpcAddress().equals(conf.getTargetConfigNode().getIp())
|| conf.getRpcPort() != conf.getTargetConfigNode().getPort())) {
throw new ConfigurationException(
- "config_nodes",
+ "target_config_nodes",
conf.getTargetConfigNode().getIp() + ":" +
conf.getTargetConfigNode().getPort(),
conf.getRpcAddress() + ":" + conf.getRpcPort());
}
@@ -160,7 +160,7 @@ public class ConfigNodeStartupCheck {
* Check if the current ConfigNode is SeedConfigNode. If true, do the
SeedConfigNode configuration
* as well.
*
- * @return True if the config_nodes points to itself
+ * @return True if the target_config_nodes points to itself
*/
private boolean isSeedConfigNode() {
boolean result =
diff --git
a/confignode/src/test/resources/confignode1conf/iotdb-confignode.properties
b/confignode/src/test/resources/confignode1conf/iotdb-confignode.properties
index c786e2021c..bedc6540f5 100644
--- a/confignode/src/test/resources/confignode1conf/iotdb-confignode.properties
+++ b/confignode/src/test/resources/confignode1conf/iotdb-confignode.properties
@@ -20,7 +20,7 @@
rpc_address=0.0.0.0
rpc_port=22277
consensus_port=22278
-config_nodes=0.0.0.0:22277
+target_config_nodes=0.0.0.0:22277
config_node_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
data_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
schema_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
diff --git
a/confignode/src/test/resources/confignode2conf/iotdb-confignode.properties
b/confignode/src/test/resources/confignode2conf/iotdb-confignode.properties
index 68854e3f4b..d9a1710a3f 100644
--- a/confignode/src/test/resources/confignode2conf/iotdb-confignode.properties
+++ b/confignode/src/test/resources/confignode2conf/iotdb-confignode.properties
@@ -20,7 +20,7 @@
rpc_address=0.0.0.0
rpc_port=22279
consensus_port=22280
-config_nodes=0.0.0.0:22277
+target_config_nodes=0.0.0.0:22277
config_node_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
data_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
schema_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
diff --git
a/confignode/src/test/resources/confignode3conf/iotdb-confignode.properties
b/confignode/src/test/resources/confignode3conf/iotdb-confignode.properties
index 79e1f871e6..37d54351d2 100644
--- a/confignode/src/test/resources/confignode3conf/iotdb-confignode.properties
+++ b/confignode/src/test/resources/confignode3conf/iotdb-confignode.properties
@@ -20,7 +20,7 @@
rpc_address=0.0.0.0
rpc_port=22281
consensus_port=22282
-config_nodes=0.0.0.0:22277
+target_config_nodes=0.0.0.0:22277
config_node_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
data_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
schema_region_consensus_protocol_class=org.apache.iotdb.consensus.ratis.RatisConsensus
diff --git a/docs/UserGuide/Cluster/Cluster-Setup.md
b/docs/UserGuide/Cluster/Cluster-Setup.md
index dbf46e8136..e6b3ef0d20 100644
--- a/docs/UserGuide/Cluster/Cluster-Setup.md
+++ b/docs/UserGuide/Cluster/Cluster-Setup.md
@@ -89,7 +89,7 @@ Important parameters in iotdb-confignode.properties:
| rpc\_address | Internal rpc service address of ConfigNode |
| rpc\_port | Internal rpc service address of ConfigNode |
| consensus\_port | ConfigNode replication consensus protocol communication
port |
-| config\_nodes | Target ConfigNode address, if the current is the first
ConfigNode, then set its address:port |
+| target\_config\_nodes | Target ConfigNode address, if the current is the
first ConfigNode, then set its address:port |
| data\_replication\_factor | Data replication factor, no more than DataNode
number |
| data\_region\_consensus\_protocol\_class | Consensus protocol of data
replicas |
| schema\_replication\_factor | Schema replication factor, no more than
DataNode number |
@@ -116,6 +116,6 @@ Important parameters in iotdb-datanode.properties
| mpp\_data\_exchange\_port | Data flow port of DataNode inside cluster
|
| data\_region\_consensus\_port | Data replicas communication port for
consensus |
| schema\_region\_consensus\_port | Schema replicas communication port for
consensus |
-| config\_nodes | Running ConfigNode of the Cluster |
+| target\_config\_nodes | Running ConfigNode of the Cluster |
More details [DataNode
Configurations](https://iotdb.apache.org/UserGuide/Master/Reference/DataNode-Config-Manual.html)
\ No newline at end of file
diff --git a/docs/UserGuide/Reference/ConfigNode-Config-Manual.md
b/docs/UserGuide/Reference/ConfigNode-Config-Manual.md
index e54d9173f9..13009ef280 100644
--- a/docs/UserGuide/Reference/ConfigNode-Config-Manual.md
+++ b/docs/UserGuide/Reference/ConfigNode-Config-Manual.md
@@ -85,9 +85,9 @@ IoTDB Cluster configuration is in ConfigNode.
|Default| 6667 |
|Effective|After restarting system|
-* config\_nodes
+* target\_config\_nodes
-|Name| config\_nodes |
+|Name| target\_config\_nodes |
|:---:|:---|
|Description| Target ConfigNode address, for current ConfigNode to join the
cluster |
|Type| String |
diff --git a/docs/UserGuide/Reference/DataNode-Config-Manual.md
b/docs/UserGuide/Reference/DataNode-Config-Manual.md
index 0164e772a7..7f62659ed1 100644
--- a/docs/UserGuide/Reference/DataNode-Config-Manual.md
+++ b/docs/UserGuide/Reference/DataNode-Config-Manual.md
@@ -247,9 +247,9 @@ The permission definitions are in
${IOTDB\_CONF}/conf/jmx.access.
|Default| 50010 |
|Effective|After restarting system|
-* config\_nodes
+* target\_config\_nodes
-|Name| config\_nodes |
+|Name| target\_config\_nodes |
|:---:|:---|
|Description| ConfigNode Address for DataNode to join cluster |
|Type| String |
diff --git a/docs/zh/UserGuide/Cluster/Cluster-Setup.md
b/docs/zh/UserGuide/Cluster/Cluster-Setup.md
index a6503aa7c1..625bbb28fc 100644
--- a/docs/zh/UserGuide/Cluster/Cluster-Setup.md
+++ b/docs/zh/UserGuide/Cluster/Cluster-Setup.md
@@ -89,7 +89,7 @@ mvn clean package -pl distribution -am -DskipTests
| rpc\_address | ConfigNode 在集群内部通讯使用的地址 |
| rpc\_port | ConfigNode 在集群内部通讯使用的端口 |
| consensus\_port | ConfigNode 副本组共识协议通信使用的端口 |
-| config\_nodes | 种子 ConfigNode 地址,第一个 ConfigNode 配置自己的 address:port
|
+| target\_config\_nodes | 种子 ConfigNode 地址,第一个 ConfigNode 配置自己的
address:port |
| data\_replication\_factor | 数据副本数,DataNode 数量不应少于此数目 |
| data\_region\_consensus\_protocol\_class | 数据副本组的共识协议 |
| schema\_replication\_factor | 元数据副本数,DataNode 数量不应少于此数目 |
@@ -116,6 +116,6 @@ iotdb-datanode.properties 中的重要配置如下
| mpp\_data\_exchange\_port | DataNode 在集群内部接收数据流使用的端口 |
| data\_region\_consensus\_port | DataNode 的数据副本间共识协议通信的端口 |
| schema\_region\_consensus\_port | DataNode 的元数据副本间共识协议通信的端口 |
-| config\_nodes | 集群中正在运行的 ConfigNode 地址 |
+| target\_config\_nodes | 集群中正在运行的 ConfigNode 地址 |
具体参考
[DataNode配置参数](https://iotdb.apache.org/zh/UserGuide/Master/Reference/DataNode-Config-Manual.html)
\ No newline at end of file
diff --git a/docs/zh/UserGuide/Reference/ConfigNode-Config-Manual.md
b/docs/zh/UserGuide/Reference/ConfigNode-Config-Manual.md
index 1472e8d1ec..fd2a4859dc 100644
--- a/docs/zh/UserGuide/Reference/ConfigNode-Config-Manual.md
+++ b/docs/zh/UserGuide/Reference/ConfigNode-Config-Manual.md
@@ -83,9 +83,9 @@ IoTDB 集群的全局配置通过 ConfigNode 配置。
|默认值| 6667 |
|改后生效方式|重启服务生效|
-* config\_nodes
+* target\_config\_nodes
-|名字| config\_nodes |
+|名字| target\_config\_nodes |
|:---:|:---|
|描述| 目标 ConfigNode 地址,ConfigNode 通过此地址加入集群 |
|类型| String |
diff --git a/docs/zh/UserGuide/Reference/DataNode-Config-Manual.md
b/docs/zh/UserGuide/Reference/DataNode-Config-Manual.md
index 45066a8270..0a08bc6907 100644
--- a/docs/zh/UserGuide/Reference/DataNode-Config-Manual.md
+++ b/docs/zh/UserGuide/Reference/DataNode-Config-Manual.md
@@ -229,9 +229,9 @@ IoTDB DataNode 与 Standalone 模式共用一套配置文件,均位于 IoTDB
|默认值| 50010 |
|改后生效方式|重启服务生效|
-* config\_nodes
+* target\_config\_nodes
-|名字| config\_nodes |
+|名字| target\_config\_nodes |
|:---:|:---|
|描述| ConfigNode 地址,DataNode 启动时通过此地址加入集群 |
|类型| String |
diff --git
a/integration-test/src/main/java/org/apache/iotdb/it/env/ConfigNodeWrapper.java
b/integration-test/src/main/java/org/apache/iotdb/it/env/ConfigNodeWrapper.java
index 05b6e51b77..468312edb7 100644
---
a/integration-test/src/main/java/org/apache/iotdb/it/env/ConfigNodeWrapper.java
+++
b/integration-test/src/main/java/org/apache/iotdb/it/env/ConfigNodeWrapper.java
@@ -50,7 +50,7 @@ public class ConfigNodeWrapper extends AbstractNodeWrapper {
properties.setProperty("rpc_address", super.getIp());
properties.setProperty("rpc_port", String.valueOf(getPort()));
properties.setProperty("consensus_port",
String.valueOf(this.consensusPort));
- properties.setProperty("config_nodes", this.targetConfigNode);
+ properties.setProperty("target_config_nodes", this.targetConfigNode);
properties.setProperty(
"config_node_consensus_protocol_class",
"org.apache.iotdb.consensus.standalone.StandAloneConsensus");
diff --git
a/integration-test/src/main/java/org/apache/iotdb/it/env/DataNodeWrapper.java
b/integration-test/src/main/java/org/apache/iotdb/it/env/DataNodeWrapper.java
index eb2f46380f..965ce1c2b3 100644
---
a/integration-test/src/main/java/org/apache/iotdb/it/env/DataNodeWrapper.java
+++
b/integration-test/src/main/java/org/apache/iotdb/it/env/DataNodeWrapper.java
@@ -54,7 +54,7 @@ public class DataNodeWrapper extends AbstractNodeWrapper {
"schema_region_consensus_port",
String.valueOf(this.schemaRegionConsensusPort));
properties.setProperty("connection_timeout_ms", "30000");
if (this.targetConfigNode != null) {
- properties.setProperty("config_nodes", this.targetConfigNode);
+ properties.setProperty("target_config_nodes", this.targetConfigNode);
}
}
diff --git a/server/src/assembly/resources/conf/iotdb-datanode.properties
b/server/src/assembly/resources/conf/iotdb-datanode.properties
index 540b419341..3393731f21 100644
--- a/server/src/assembly/resources/conf/iotdb-datanode.properties
+++ b/server/src/assembly/resources/conf/iotdb-datanode.properties
@@ -59,14 +59,14 @@ data_region_consensus_port=40010
# port for consensus's communication for schema region between cluster nodes.
schema_region_consensus_port=50010
-# Used for connecting to the ConfigNodeGroup
+# At least one running ConfigNode should be set for joining the cluster
# Format: ip:port
# where the ip should be consistent with the target ConfigNode's
confignode_rpc_address,
# and the port should be consistent with the target ConfigNode's
confignode_rpc_port.
# When successfully connecting to the ConfigNodeGroup, DataNode will get all
online
# config nodes and store them in memory.
# Datatype: String
-config_nodes=127.0.0.1:22277
+target_config_nodes=127.0.0.1:22277
# Datatype: boolean
# rpc_thrift_compression_enable=false
diff --git a/server/src/main/java/org/apache/iotdb/db/conf/IoTDBDescriptor.java
b/server/src/main/java/org/apache/iotdb/db/conf/IoTDBDescriptor.java
index eb4280dfca..94fbd09147 100644
--- a/server/src/main/java/org/apache/iotdb/db/conf/IoTDBDescriptor.java
+++ b/server/src/main/java/org/apache/iotdb/db/conf/IoTDBDescriptor.java
@@ -1645,7 +1645,7 @@ public class IoTDBDescriptor {
}
public void loadClusterProps(Properties properties) {
- String configNodeUrls = properties.getProperty("config_nodes");
+ String configNodeUrls = properties.getProperty("target_config_nodes");
if (configNodeUrls != null) {
try {
conf.setTargetConfigNodeList(NodeUrlUtils.parseTEndPointUrls(configNodeUrls));
diff --git a/server/src/test/resources/datanode1conf/iotdb-datanode.properties
b/server/src/test/resources/datanode1conf/iotdb-datanode.properties
index 876e388cbe..cd17728791 100644
--- a/server/src/test/resources/datanode1conf/iotdb-datanode.properties
+++ b/server/src/test/resources/datanode1conf/iotdb-datanode.properties
@@ -26,7 +26,7 @@ internal_port=9003
data_region_consensus_port=40010
schema_region_consensus_port=50030
-config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
+target_config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
system_dir=target/datanode1/system
data_dirs=target/datanode1/data
diff --git a/server/src/test/resources/datanode2conf/iotdb-datanode.properties
b/server/src/test/resources/datanode2conf/iotdb-datanode.properties
index 0336c05984..097e063007 100644
--- a/server/src/test/resources/datanode2conf/iotdb-datanode.properties
+++ b/server/src/test/resources/datanode2conf/iotdb-datanode.properties
@@ -26,7 +26,7 @@ internal_port=9005
data_region_consensus_port=40012
schema_region_consensus_port=50032
-config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
+target_config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
system_dir=target/datanode2/system
data_dirs=target/datanode2/data
diff --git a/server/src/test/resources/datanode3conf/iotdb-datanode.properties
b/server/src/test/resources/datanode3conf/iotdb-datanode.properties
index 6969498b33..86be90b040 100644
--- a/server/src/test/resources/datanode3conf/iotdb-datanode.properties
+++ b/server/src/test/resources/datanode3conf/iotdb-datanode.properties
@@ -26,7 +26,7 @@ internal_port=9007
data_region_consensus_port=40014
schema_region_consensus_port=50034
-config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
+target_config_nodes=0.0.0.0:22277,0.0.0.0:22279,0.0.0.0:22281
system_dir=target/datanode3/system
data_dirs=target/datanode3/data