Myasuka commented on a change in pull request #16582:
URL: https://github.com/apache/flink/pull/16582#discussion_r677214452
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskStateManager.java
##########
@@ -42,7 +42,7 @@
* <p>This interface also offers the complementary method that provides access
to previously saved
* state of operator instances in the task for restore purposes.
*/
-public interface TaskStateManager extends CheckpointListener, AutoCloseable {
+public interface TaskStateManager extends InternalCheckpointListener,
AutoCloseable {
Review comment:
I previously thought that `TaskStateManagerImpl` could also leverage the
`notifyCheckpointSubsumed` API to help remove local states. This is actually
another story and not seems so valuable in current implementation.
I will remove this in next commit.
--
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]
