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

    https://github.com/apache/spark/pull/19717#discussion_r154807233
  
    --- 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 --
    
    I don't think it's absolutely necessary to have a parameter for the 
namespace. `--kubernetes-namespace` is not significantly shorter than 
`spark.kubernetes.namespace`. @mccheah @foxish WDYT?  


---

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

Reply via email to