diff --git a/flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java b/flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java index 1c20393a3c35d..265cb5ca551db 100644 --- a/flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java +++ b/flink-streaming-java/src/main/java/org/apache/flink/streaming/runtime/tasks/StreamTask.java @@ -902,8 +902,15 @@ public void run() { List operatorStatesStream = new ArrayList<>(snapshotInProgressList.size()); for (OperatorSnapshotResult snapshotInProgress : snapshotInProgressList) { - operatorStatesBackend.add(FutureUtil.runIfNotDoneAndGet(snapshotInProgress.getOperatorStateManagedFuture())); - operatorStatesStream.add(FutureUtil.runIfNotDoneAndGet(snapshotInProgress.getOperatorStateRawFuture())); + if (null != snapshotInProgress) { + operatorStatesBackend.add( + FutureUtil.runIfNotDoneAndGet(snapshotInProgress.getOperatorStateManagedFuture())); + operatorStatesStream.add( + FutureUtil.runIfNotDoneAndGet(snapshotInProgress.getOperatorStateRawFuture())); + } else { + operatorStatesBackend.add(null); + operatorStatesStream.add(null); + } } final long asyncEndNanos = System.nanoTime(); @@ -950,7 +957,9 @@ public void run() { public void close() { // cleanup/release ongoing snapshot operations for (OperatorSnapshotResult snapshotResult : snapshotInProgressList) { - snapshotResult.cancel(); + if (null != snapshotResult) { + snapshotResult.cancel(); + } } } } @@ -995,14 +1004,7 @@ public void executeCheckpointing() throws Exception { try { for (StreamOperator op : allOperators) { - - createStreamFactory(op); - snapshotNonPartitionableState(op); - - OperatorSnapshotResult snapshotInProgress = - op.snapshotState(checkpointMetaData.getCheckpointId(), checkpointMetaData.getTimestamp(), streamFactory); - - snapshotInProgressList.add(snapshotInProgress); + checkpointStreamOperator(op); } if (LOG.isDebugEnabled()) { @@ -1029,7 +1031,9 @@ public void executeCheckpointing() throws Exception { if (failed) { // Cleanup to release resources for (OperatorSnapshotResult operatorSnapshotResult : snapshotInProgressList) { - operatorSnapshotResult.cancel(); + if (null != operatorSnapshotResult) { + operatorSnapshotResult.cancel(); + } } if (LOG.isDebugEnabled()) { @@ -1039,7 +1043,24 @@ public void executeCheckpointing() throws Exception { } } } + } + + private void checkpointStreamOperator(StreamOperator op) throws Exception { + if (null != op) { + createStreamFactory(op); + snapshotNonPartitionableState(op); + + OperatorSnapshotResult snapshotInProgress = op.snapshotState( + checkpointMetaData.getCheckpointId(), + checkpointMetaData.getTimestamp(), + streamFactory); + snapshotInProgressList.add(snapshotInProgress); + } else { + nonPartitionedStates.add(null); + OperatorSnapshotResult emptySnapshotInProgress = new OperatorSnapshotResult(); + snapshotInProgressList.add(emptySnapshotInProgress); + } } private void createStreamFactory(StreamOperator operator) throws IOException {