AHeise commented on a change in pull request #11507: [FLINK-16587] Add basic CheckpointBarrierHandler for unaligned checkpoint URL: https://github.com/apache/flink/pull/11507#discussion_r402567144
########## File path: flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java ########## @@ -1096,14 +1134,18 @@ void handleAsyncException(String message, Throwable exception) { private final AtomicReference<CheckpointingOperation.AsyncCheckpointState> asyncCheckpointState = new AtomicReference<>( CheckpointingOperation.AsyncCheckpointState.RUNNING); + private final Future<?> bufferPersistingFuture; Review comment: Hm the type is completely irrelevant. Any reason to change it? I actually haven't really understood, why the future returns the stateobjects to begin with (but Roman verified that all writing has already been done). ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services