zentol commented on a change in pull request #15348: URL: https://github.com/apache/flink/pull/15348#discussion_r602125372
########## File path: flink-runtime/src/main/java/org/apache/flink/runtime/scheduler/adaptive/AdaptiveScheduler.java ########## @@ -860,6 +902,7 @@ private ExecutionGraph createExecutionGraphAndRestoreState(JobGraph adjustedJobG TaskDeploymentDescriptorFactory.PartitionLocationConstraint.MUST_BE_KNOWN, initializationTimestamp, vertexAttemptNumberStore, + computeVertexParallelismStore(adjustedJobGraph), Review comment: Actually, if we were to base the maxP on the original-original JobGraph (the one in the DefaultJobManagerRunnerFactory), then that would also work, and be most in line with existing behavior. -- 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