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

Reply via email to