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


Reply via email to