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


##########
modules/partition-replicator/src/main/java/org/apache/ignite/internal/partition/replicator/raft/ZonePartitionRaftListener.java:
##########
@@ -412,6 +412,20 @@ public void onShutdown() {
         }
     }
 
+    @Override
+    public long getPersistedAppliedIndex() {

Review Comment:
   IGNITE-28216 fixes the known root cause: specific missed lastApplied() bumps 
in partition processing. This PR adds a detection mechanism: if truncation 
below applied index ever happens (from this bug or any future bug), we fail the 
node instead of silently corrupting data.
   
   As a result these are complementary. IGNITE-28216 prevents the gap; our PR 
catches it if it ever occurs anyway, due to some other unknown bug or nuance -- 
a similar future bug would again cause silent corruption with no indication of 
something went wrong.



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