clownxc commented on code in PR #22555:
URL: https://github.com/apache/flink/pull/22555#discussion_r1190120652
##########
flink-runtime/src/main/java/org/apache/flink/runtime/scheduler/adaptivebatch/DefaultVertexParallelismAndInputInfosDecider.java:
##########
@@ -546,7 +553,7 @@ static DefaultVertexParallelismAndInputInfosDecider from(
configuration.get(
BatchExecutionOptions.ADAPTIVE_AUTO_PARALLELISM_AVG_DATA_VOLUME_PER_TASK),
configuration.get(
- BatchExecutionOptions
-
.ADAPTIVE_AUTO_PARALLELISM_DEFAULT_SOURCE_PARALLELISM));
+
BatchExecutionOptions.ADAPTIVE_AUTO_PARALLELISM_DEFAULT_SOURCE_PARALLELISM),
+ configuration.get(CoreOptions.DEFAULT_PARALLELISM));
}
Review Comment:
> parallelism.default should be obtained from ExecutionConfig instead of
jobmaster Configuration
Thank you very much for review. I want to get the parallelism of
ExecutionConfig as the default value.
```java
public int getParallelism() {
return configuration.get(CoreOptions.DEFAULT_PARALLELISM);
}
```
--
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.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
