AHeise commented on a change in pull request #13735: URL: https://github.com/apache/flink/pull/13735#discussion_r517717510
########## File path: flink-runtime/src/main/java/org/apache/flink/runtime/checkpoint/StateAssignmentOperation.java ########## @@ -620,18 +758,112 @@ private static void checkStateMappingCompleteness( chainOpParallelStates, oldParallelism, newParallelism); + } + + static class TaskStateAssignment { Review comment: I split the two pass commit into two, but they are strongly related, so I'm not sure if the should exist on their own. ---------------------------------------------------------------- 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