diff options
| author | Marcelo Vanzin <vanzin@cloudera.com> | 2016-05-10 10:35:54 -0700 |
|---|---|---|
| committer | Marcelo Vanzin <vanzin@cloudera.com> | 2016-05-10 10:35:54 -0700 |
| commit | 0b9cae42426e14060bc6182c037fd715f35a2d23 (patch) | |
| tree | fbf356424566ce8bd027fa034f0be6e58ce73066 /yarn | |
| parent | 36c5892b4631d1d5d70e569fce12efedd82bdef0 (diff) | |
| download | spark-0b9cae42426e14060bc6182c037fd715f35a2d23.tar.gz spark-0b9cae42426e14060bc6182c037fd715f35a2d23.tar.bz2 spark-0b9cae42426e14060bc6182c037fd715f35a2d23.zip | |
[SPARK-11249][LAUNCHER] Throw error if app resource is not provided.
Without this, the code would build an invalid spark-submit command line,
and a more cryptic error would be presented to the user. Also, expose
a constant that allows users to set a dummy resource in cases where
they don't need an actual resource file; for backwards compatibility,
that uses the same "spark-internal" resource that Spark itself uses.
Tested via unit tests, run-example, spark-shell, and running the
thrift server with mixed spark and hive command line arguments.
Author: Marcelo Vanzin <vanzin@cloudera.com>
Closes #12909 from vanzin/SPARK-11249.
Diffstat (limited to 'yarn')
| -rw-r--r-- | yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala b/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala index b2b4d84f53..7df11ca760 100644 --- a/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala +++ b/yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnClusterSuite.scala @@ -147,7 +147,7 @@ class YarnClusterSuite extends BaseYarnClusterSuite { .setPropertiesFile(propsFile) .setMaster("yarn") .setDeployMode("client") - .setAppResource("spark-internal") + .setAppResource(SparkLauncher.NO_RESOURCE) .setMainClass(mainClassName(YarnLauncherTestApp.getClass)) .startApplication() |