Repository: spark
Updated Branches:
  refs/heads/master 0494c80ef -> c986e933a


[SPARK-10711] [SPARKR] Do not assume spark.submit.deployMode is always set

In ```RUtils.sparkRPackagePath()``` we
1. Call ``` sys.props("spark.submit.deployMode")``` which returns null if 
```spark.submit.deployMode``` is not suet
2. Call ``` sparkConf.get("spark.submit.deployMode")``` which throws 
```NoSuchElementException``` if ```spark.submit.deployMode``` is not set. This 
patch simply passes a default value ("cluster") for 
```spark.submit.deployMode```.

cc rxin

Author: Hossein <[email protected]>

Closes #8832 from falaki/SPARK-10711.


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/c986e933
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/c986e933
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/c986e933

Branch: refs/heads/master
Commit: c986e933a900602af47966bd41edb2116c421a39
Parents: 0494c80
Author: Hossein <[email protected]>
Authored: Mon Sep 21 21:09:59 2015 -0700
Committer: Reynold Xin <[email protected]>
Committed: Mon Sep 21 21:09:59 2015 -0700

----------------------------------------------------------------------
 core/src/main/scala/org/apache/spark/api/r/RUtils.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/c986e933/core/src/main/scala/org/apache/spark/api/r/RUtils.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/api/r/RUtils.scala 
b/core/src/main/scala/org/apache/spark/api/r/RUtils.scala
index 9e807cc..fd5646b 100644
--- a/core/src/main/scala/org/apache/spark/api/r/RUtils.scala
+++ b/core/src/main/scala/org/apache/spark/api/r/RUtils.scala
@@ -44,7 +44,7 @@ private[spark] object RUtils {
         (sys.props("spark.master"), sys.props("spark.submit.deployMode"))
       } else {
         val sparkConf = SparkEnv.get.conf
-        (sparkConf.get("spark.master"), 
sparkConf.get("spark.submit.deployMode"))
+        (sparkConf.get("spark.master"), 
sparkConf.get("spark.submit.deployMode", "client"))
       }
 
     val isYarnCluster = master != null && master.contains("yarn") && 
deployMode == "cluster"


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to