Github user adambalogh commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r215947975
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
Github user adambalogh commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r214428426
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r214414143
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
Github user adambalogh commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r214401744
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
Github user ifilonenko commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r214381467
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/22289#discussion_r214233802
--- Diff:
launcher/src/main/java/org/apache/spark/launcher/AbstractCommandBuilder.java ---
@@ -200,6 +200,7 @@ void addOptionString(List cmd, String
GitHub user adambalogh opened a pull request:
https://github.com/apache/spark/pull/22289
[SPARK-25200][YARN] Allow specifying HADOOP_CONF_DIR as spark property
## What changes were proposed in this pull request?
When submitting applications to Yarn in cluster mode, using
