tsunghanjacktsai commented on code in PR #5685:
URL: https://github.com/apache/rocketmq/pull/5685#discussion_r1046873303
##########
common/src/main/java/org/apache/rocketmq/common/ControllerConfig.java:
##########
@@ -59,6 +59,11 @@ public class ControllerConfig {
*/
private volatile boolean notifyBrokerRoleChanged = true;
+ /**
+ * How many DLedger logs are committed to trigger a snapshot
+ */
+ private int statemachineSnapshotThreshold = 1000;
Review Comment:
There's also a parameter called "maxSnapshotReservedNum", which is for
setting the max number of snapshot files keeping within the store. I believe it
should be added to the controller config as well.
--
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]
