Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5239#discussion_r162583920
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskExecutor.java
---
@@ -380,6 +390,20 @@ public void postStop() throws Exception {
ResultPartitionConsumableNotifier
resultPartitionConsumableNotifier =
jobManagerConnection.getResultPartitionConsumableNotifier();
PartitionProducerStateChecker partitionStateChecker =
jobManagerConnection.getPartitionStateChecker();
+ final TaskLocalStateStore localStateStore =
localStateStoresManager.localStateStoreForTask(
+ jobId,
+ taskInformation.getJobVertexId(),
+ tdd.getSubtaskIndex());
+
+ final JobManagerTaskRestore taskRestore =
tdd.getTaskRestore();
--- End diff --
In general a good idea, but I would make it a separate change because it
was not introduced or touched by this PR.
---
