[GitHub] [spark] srowen commented on a change in pull request #29037: [SPARK-32224][CORE] Add a simpler way to pass 'spark.yarn.priority' through the SparkSubmit

[GitBox <gi...@apache.org>]

srowen commented on a change in pull request #29037:
URL: https://github.com/apache/spark/pull/29037#discussion_r451939730



##########
File path: core/src/main/scala/org/apache/spark/deploy/SparkSubmitArguments.scala
##########
@@ -562,6 +570,7 @@ private[deploy] class SparkSubmitArguments(args: Seq[String], env: Map[String, S
         |
         | Spark on YARN only:
         |  --queue QUEUE_NAME          The YARN queue to submit to (Default: "default").
+        |  --priority PRIORITY         The priority of your YARN application.

Review comment:
       Wouldn't this extend to lots of other arguments? I think the ones that exist are very common and widely applicable. There are probably a few too many already.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org