[GitHub] flink pull request #5578: [FLINK-8777][state]Improve resource release for lo...

[StefanRRichter]

Github user StefanRRichter commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5578#discussion_r170975679
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskLocalStateStoreImpl.java
 ---
    @@ -159,6 +166,11 @@ public TaskStateSnapshot retrieveLocalState(long 
checkpointID) {
                TaskStateSnapshot snapshot;
                synchronized (lock) {
                        snapshot = 
storedTaskStateByCheckpointID.get(checkpointID);
    +
    +                   if (retrieveWithDiscard) {
    +                           // Only the TaskStateSnapshot.checkpointID == 
checkpointID is useful, we remove the others
    --- End diff --
    
    Comment is no longer required.


---