[
https://issues.apache.org/jira/browse/FLINK-6058?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16077495#comment-16077495
]
ASF GitHub Bot commented on FLINK-6058:
---------------------------------------
Github user zjureel commented on the issue:
https://github.com/apache/flink/pull/3921
@aljoscha Sorry for replying late, your suggestion sounds good to me. In
fact, I was hesitating to add `getDefaultParallelism()` in `ContextEnvironment`
and `ClusterClient` for there have been `getParallelism()` in
`ContextEnvironment`. Thank you for your suggestion, I will fix it as you
suggested soon :)
> Don't read DEFAULT_PARALLELISM from GlobalConfiguration
> -------------------------------------------------------
>
> Key: FLINK-6058
> URL: https://issues.apache.org/jira/browse/FLINK-6058
> Project: Flink
> Issue Type: Improvement
> Components: DataStream API
> Reporter: Aljoscha Krettek
> Assignee: Fang Yong
>
> In the constructor of {{StreamContextEnvironment}} we read the
> {{DEFAULT_PARALLELISM}} from the {{GlobalConfiguration}}. This assumes that
> the environment variables are correctly set and can lead to problems. We
> should read the default parallelism in the client and set it in the
> {{ContextEnvironment}} that it creates. This can then be read by the
> {{StreamContextEnvironment}}.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
