fredia commented on code in PR #19907:
URL: https://github.com/apache/flink/pull/19907#discussion_r895604615
##########
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskLocalStateStore.java:
##########
@@ -76,7 +74,7 @@ public interface TaskLocalStateStore {
/**
* Remove all checkpoints from the store that match the given predicate.
*
- * @param matcher the predicate that selects the checkpoints for pruning.
+ * @param checkpointID the checkpoints for pruning.
*/
- void pruneMatchingCheckpoints(LongPredicate matcher);
+ void pruneNotMatchingCheckpoints(long checkpointID);
Review Comment:
It is not a pure refactoring.
This method is called by `TaskStateManagerImpl#prioritizedOperatorState()`
to prune other checkpoint when restoring. For changelog, the materialized part
shouldn't be pruned when restoring, the `@param checkpointID` is used to find
the reference between checkpoints.
--
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]
