[
https://issues.apache.org/jira/browse/FLINK-6222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16555404#comment-16555404
]
Dawid Wysakowicz commented on FLINK-6222:
-----------------------------------------
Fixed in master via: 81ff3c45004cd6e3cc5b0f75dc567f5f57d1f1ed
4c4e3d66c9635e523c8ab1539f635ef1dbb24e61
Fixed in 1.6 via: 34641a8987b937a8bfeb7a5e1c0ffa918f9f88d7
2194f473f376e50e2ac354357475f984f120404f
> YARN: setting environment variables in an easier fashion
> --------------------------------------------------------
>
> Key: FLINK-6222
> URL: https://issues.apache.org/jira/browse/FLINK-6222
> Project: Flink
> Issue Type: Improvement
> Components: Startup Shell Scripts
> Affects Versions: 1.2.0
> Environment: YARN, EMR
> Reporter: Craig Foster
> Assignee: Dawid Wysakowicz
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.6.0, 1.7.0
>
> Attachments: patch0-add-yarn-hadoop-conf.diff
>
>
> Right now we require end-users to set YARN_CONF_DIR or HADOOP_CONF_DIR and
> sometimes FLINK_CONF_DIR.
> For example, in [1], it is stated:
> “Please note that the Client requires the YARN_CONF_DIR or HADOOP_CONF_DIR
> environment variable to be set to read the YARN and HDFS configuration.”
> In BigTop, we set this with /etc/flink/default and then a wrapper is created
> to source that. However, this is slightly cumbersome and we don't have a
> central place within the Flink project itself to source environment
> variables. config.sh could do this but it doesn't have information about
> FLINK_CONF_DIR. For YARN and Hadoop variables, I already have a solution that
> would add "env.yarn.confdir" and "env.hadoop.confdir" variables to the
> flink-conf.yaml file and then we just symlink /etc/lib/flink/conf/ and
> /etc/flink/conf.
> But we could also add a flink-env.sh file to set these variables and decouple
> them from config.sh entirely.
> I'd like to know the opinion/preference of others and what would be more
> amenable.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)