Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19717#discussion_r154749355
  
    --- Diff: 
core/src/main/scala/org/apache/spark/deploy/SparkSubmitArguments.scala ---
    @@ -590,6 +600,11 @@ private[deploy] class SparkSubmitArguments(args: 
Seq[String], env: Map[String, S
             |                              the node running the Application 
Master via the Secure
             |                              Distributed Cache, for renewing the 
login tickets and the
             |                              delegation tokens periodically.
    +        |
    +        | Kubernetes only:
    +        |  --kubernetes-namespace NS   The namespace in the Kubernetes 
cluster within which the
    --- End diff --
    
    Is it a big deal to not add this as a command line arg and force people to 
use the configuration instead? I'd prefer to not add even more cluster-specific 
switches to `SparkSubmit`, at least not until it is refactored to be pluggable.


---

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

Reply via email to