Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16803#discussion_r99583052
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -480,7 +479,12 @@ object SparkSubmit extends CommandLineUtils {
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16803#discussion_r99528722
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -480,7 +479,12 @@ object SparkSubmit extends CommandLineUtils {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16803#discussion_r99518229
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitUtilsSuite.scala ---
@@ -258,4 +259,53 @@ class SparkSubmitUtilsSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16803#discussion_r99518186
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -480,7 +479,12 @@ object SparkSubmit extends CommandLineUtils {