[GitHub] flink pull request #5239: [FLINK-8360] Implement task-local state recovery

[pnowojski]

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

    https://github.com/apache/flink/pull/5239#discussion_r159866220
  
    --- Diff: 
flink-streaming-java/src/test/java/org/apache/flink/streaming/runtime/io/BarrierBufferTest.java
 ---
    @@ -1483,11 +1482,6 @@ long getLastReportedBytesBufferedInAlignment() {
                        return lastReportedBytesBufferedInAlignment;
                }
     
    -           @Override
    --- End diff --
    
    I think you could replace `ValidatingCheckpointHandler` with 
`CheckpointSequenceValidator` and remove this code duplication.


---