Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/1349#discussion_r14742342
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -27,25 +27,39 @@ import org.apache.spark.executor.ExecutorURLClassLoader
import org.apache.spark.util.Utils
/**
- * Scala code behind the spark-submit script. The script handles setting
up the classpath with
- * relevant Spark dependencies and provides a layer over the different
cluster managers and deploy
- * modes that Spark supports.
+ * Main gateway of launching a Spark application.
+ *
+ * This script handles setting up the classpath with relevant Spark
dependencies and provides
--- End diff --
I think saying "this script" is a little confusing. If the class
encapsulated all the logic behind the script, it might make a little more
sense, but the script has some logic of its own.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
