GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/16695
[SPARK-19277][yarn] Localize topology scripts inside Hadoop configuration.
Hadoop has some configurations that can be used to run an external script
that prints out information about cluster host topology. When a Spark
application runs, the Hadoop configuration is localized in each Spark
container, but these paths remain pointing to their original location,
which may not exist in the target machine.
This change updates those configs so that the Spark AM points to the
localized path of the scripts, when the original files are localized
by Spark (i.e. when they live inside the Hadoop configuration directory).
The scripts seem to only be used by the AM, so the change skips updating
the configs for executors, since that would be more complicated.
Tested with added unit test and by trying this out in a real cluster and
verifying the updated configuration was being used.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vanzin/spark SPARK-19277
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/16695.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #16695
----
commit df3922ef023570846da6d86d121fe50be969adbd
Author: Marcelo Vanzin <van...@cloudera.com>
Date: 2017-01-24T00:50:23Z
[SPARK-19277][yarn] Localize topology scripts inside Hadoop configuration.
Hadoop has some configurations that can be used to run an external script
that prints out information about cluster host topology. When a Spark
application runs, the Hadoop configuration is localized in each Spark
container, but these paths remain pointing to their original location,
which may not exist in the target machine.
This change updates those configs so that the Spark AM points to the
localized path of the scripts, when the original files are localized
by Spark (i.e. when they live inside the Hadoop configuration directory).
The scripts seem to only be used by the AM, so the change skips updating
the configs for executors, since that would be more complicated.
Tested with added unit test and by trying this out in a real cluster and
verifying the updated configuration was being used.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
