[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_r171177343
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskLocalStateStoreImpl.java
 ---
    @@ -300,6 +295,34 @@ private void deleteDirectory(File directory) throws 
IOException {
                }
        }
     
    +   /**
    +    * Pruning the useless checkpoints, it should be called only when 
holding the {@link #lock}.
    +    */
    +   private void pruneCheckpoints(Predicate<Long> pruningChecker, boolean 
breakOnceCheckerFalse) {
    --- End diff --
    
    We can use `LongPredicate` instead of `Predicate<Long>`


---