rpuch commented on code in PR #1331:
URL: https://github.com/apache/ignite-3/pull/1331#discussion_r1021373516
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/PartitionListener.java:
##########
@@ -327,15 +383,23 @@ public void onSnapshotSave(Path path, Consumer<Throwable>
doneClo) {
// 4) When we try to restore data starting from the minimal
lastAppliedIndex, we come to the situation
// that a raft node doesn't have such data, because the
truncation until the maximal lastAppliedIndex from 1) has happened.
// 5) Node cannot finish local recovery.
- long maxLastAppliedIndex = Math.max(storage.lastAppliedIndex(),
txStateStorage.lastAppliedIndex());
+ long maxLastAppliedIndex;
+ long maxLastAppliedTerm;
Review Comment:
Looks like we can as term must be monothonic for committed log entries (and
here we only have committed ones).
Applied this change in all 3 places.
--
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]
