ibessonov commented on code in PR #7694:
URL: https://github.com/apache/ignite-3/pull/7694#discussion_r3049612883


##########
modules/raft-api/src/main/java/org/apache/ignite/internal/raft/service/RaftGroupListener.java:
##########
@@ -96,4 +96,14 @@ default void onConfigurationCommitted(
      * Invoked once after a raft node has been shut down.
      */
     void onShutdown();
+
+    /**
+     * Returns the last applied index persisted by the state machine.
+     * Called during {@code NodeImpl.init()} to prevent truncation of 
already-applied log entries.
+     *
+     * @return persisted applied index, or 0 if unknown.
+     */
+    default long getPersistedAppliedIndex() {
+        return 0;

Review Comment:
   I believe that the real answer is "no, we should not". MG and CMG only use 
one data storage per FSM, which always guarantees that `min(lastAppliedIndex)` 
matches `max(lastAppliedIndex)`, and the corresponding troubled behavior is 
impossible. This is a distribution-zone-exclusive problem.



-- 
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]

Reply via email to