Skip to content

Conversation

@andrewor14
Copy link
Contributor

SPARK_DRIVER_MEMORY was only used to start the SparkSubmit JVM, which becomes the driver only in client mode but not cluster mode. In cluster mode, this property is simply not propagated to the worker nodes.

SPARK_EXECUTOR_MEMORY is picked up from SparkContext, but in cluster mode the driver runs on one of the worker machines, where this environment variable may not be set.

@SparkQA
Copy link

SparkQA commented Sep 23, 2014

QA tests have started for PR 2500 at commit 6217b38.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 23, 2014

QA tests have finished for PR 2500 at commit 6217b38.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Sep 23, 2014

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/20677/

@andrewor14
Copy link
Contributor Author

Tested single-node standalone mode locally. Merging into master and 1.1.

@asfgit asfgit closed this in b3fef50 Sep 23, 2014
asfgit pushed a commit that referenced this pull request Sep 23, 2014
`SPARK_DRIVER_MEMORY` was only used to start the `SparkSubmit` JVM, which becomes the driver only in client mode but not cluster mode. In cluster mode, this property is simply not propagated to the worker nodes.

`SPARK_EXECUTOR_MEMORY` is picked up from `SparkContext`, but in cluster mode the driver runs on one of the worker machines, where this environment variable may not be set.

Author: Andrew Or <andrewor14@gmail.com>

Closes #2500 from andrewor14/memory-env-vars and squashes the following commits:

6217b38 [Andrew Or] Respect SPARK_*_MEMORY for cluster mode

Conflicts:
	core/src/main/scala/org/apache/spark/deploy/SparkSubmitArguments.scala
@andrewor14 andrewor14 deleted the memory-env-vars branch September 23, 2014 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants