diff options
author | WangTaoTheTonic <barneystinson@aliyun.com> | 2014-09-18 12:07:24 -0700 |
---|---|---|
committer | Patrick Wendell <pwendell@gmail.com> | 2014-09-18 12:07:53 -0700 |
commit | 471e6a3a47bd4b94878798f6f6fc93e2e672efff (patch) | |
tree | 262759ce4d231dcaabb0efaf6d8b0dcd995186b1 | |
parent | 6cab838b9803e3294c07bbf731c47154ec57afc0 (diff) | |
download | spark-471e6a3a47bd4b94878798f6f6fc93e2e672efff.tar.gz spark-471e6a3a47bd4b94878798f6f6fc93e2e672efff.tar.bz2 spark-471e6a3a47bd4b94878798f6f6fc93e2e672efff.zip |
[SPARK-3589][Minor]remove redundant code
https://issues.apache.org/jira/browse/SPARK-3589
"export CLASSPATH" in spark-class is redundant since same variable is exported before.
We could reuse defined value "isYarnCluster" in SparkSubmit.scala.
Author: WangTaoTheTonic <barneystinson@aliyun.com>
Closes #2445 from WangTaoTheTonic/removeRedundant and squashes the following commits:
6fb6872 [WangTaoTheTonic] remove redundant code
-rw-r--r-- | core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala b/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala index 5d15af1326..3dd1dd5b82 100644 --- a/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala +++ b/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala @@ -261,7 +261,7 @@ object SparkSubmit { } // In yarn-cluster mode, use yarn.Client as a wrapper around the user class - if (clusterManager == YARN && deployMode == CLUSTER) { + if (isYarnCluster) { childMainClass = "org.apache.spark.deploy.yarn.Client" if (args.primaryResource != SPARK_INTERNAL) { childArgs += ("--jar", args.primaryResource) |