-
Notifications
You must be signed in to change notification settings - Fork 28.6k
SPARK-1680: use configs for specifying environment variables on YARN #1512
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
QA tests have started for PR 1512. This patch merges cleanly. |
QA results for PR 1512: |
I need to update so the documentation shows up properly |
QA tests have started for PR 1512. This patch merges cleanly. |
QA tests have started for PR 1512. This patch merges cleanly. |
QA results for PR 1512: |
@pwendell are you ok with exposing the spark.executorEnv. config? |
@tgravescs I think it's fine to expose that. We actually already document it in the comments of SparkConf.setExecutorEnv and such. |
Note that this also documents spark.executorEnv.* which to me means its public. If we don't want that please speak up. Author: Thomas Graves <tgraves@apache.org> Closes #1512 from tgravescs/SPARK-1680 and squashes the following commits: 11525df [Thomas Graves] more doc changes 553bad0 [Thomas Graves] fix documentation 152bf7c [Thomas Graves] fix docs 5382326 [Thomas Graves] try fix docs 32f86a4 [Thomas Graves] use configs for specifying environment variables on YARN (cherry picked from commit 41e0a21) Signed-off-by: Thomas Graves <tgraves@apache.org>
Note that this also documents spark.executorEnv.* which to me means its public. If we don't want that please speak up. Author: Thomas Graves <tgraves@apache.org> Closes apache#1512 from tgravescs/SPARK-1680 and squashes the following commits: 11525df [Thomas Graves] more doc changes 553bad0 [Thomas Graves] fix documentation 152bf7c [Thomas Graves] fix docs 5382326 [Thomas Graves] try fix docs 32f86a4 [Thomas Graves] use configs for specifying environment variables on YARN
Note that this also documents spark.executorEnv.* which to me means its public. If we don't want that please speak up.