CRZbulabula commented on code in PR #12232:
URL: https://github.com/apache/iotdb/pull/12232#discussion_r1537534426
##########
iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/client/DataNodeRequestType.java:
##########
@@ -32,7 +32,6 @@ public enum DataNodeRequestType {
STOP_REPAIR_DATA,
LOAD_CONFIGURATION,
SET_SYSTEM_STATUS,
- BROADCAST_LATEST_CONFIG_NODE_GROUP,
Review Comment:
We could keep the BROADCAST_LATEST_CONFIG_NODE_GROUP interface, since it's
never too much to broadcast immediately when the cluster ConfigNode group has
been updated. Therefore the services that rely on ConfigNodeList will save the
1s heartbeat interval time. The heartbeat function should be took as a security
method to ensure the DataNodes can update their ConfigNode list when
BROADCAST_LATEST_CONFIG_NODE_GROUP failed.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]