-
Notifications
You must be signed in to change notification settings - Fork 997
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IoTConsensusV2: Introduce IoTConsensusV2 and corresponding mode option #13440
base: master
Are you sure you want to change the base?
Conversation
iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/conf/SystemPropertiesUtils.java
Outdated
Show resolved
Hide resolved
iotdb-protocol/thrift-confignode/src/main/thrift/confignode.thrift
Outdated
Show resolved
Hide resolved
iotdb-core/node-commons/src/assembly/resources/conf/iotdb-system.properties.template
Show resolved
Hide resolved
iotdb-core/consensus/src/main/java/org/apache/iotdb/consensus/config/PipeConsensusConfig.java
Outdated
Show resolved
Hide resolved
@@ -154,10 +154,20 @@ data_replication_factor=1 | |||
# 1. org.apache.iotdb.consensus.simple.SimpleConsensus (The data_replication_factor can only be set to 1) | |||
# 2. org.apache.iotdb.consensus.iot.IoTConsensus | |||
# 3. org.apache.iotdb.consensus.ratis.RatisConsensus | |||
# effectiveMode: first_start | |||
# 4. org.apache.iotdb.consensus.ratis.IoTConsensusV2 | |||
# effectiveMode: restart |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
first_start
# Datatype: string | ||
data_region_consensus_protocol_class=org.apache.iotdb.consensus.iot.IoTConsensus | ||
|
||
# IoTConsensusV2 mode. | ||
# This parameter is unmodifiable after ConfigNode starts for the first time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this line?
# These consensus protocol modes are currently supported: | ||
# 1. batch | ||
# 2. stream | ||
# effectiveMode: first_start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
restart?
BTW, Should we allow some nodes to stream and some nodes to batch? @SteveYurongSu
@@ -1804,12 +1814,12 @@ schema_region_ratis_periodic_snapshot_interval=86400 | |||
data_region_ratis_periodic_snapshot_interval=86400 | |||
|
|||
#################### | |||
### Fast IoTConsensus Configuration | |||
### IoTConsensusV2 Configuration | |||
#################### | |||
# Default event buffer size for connector and receiver in pipe consensus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove pipeconsensus
as title.