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

    https://github.com/apache/spark/pull/19717#discussion_r154568773
  
    --- Diff: 
core/src/main/scala/org/apache/spark/deploy/SparkSubmitArguments.scala ---
    @@ -590,6 +604,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
    +        |                              application must be launched. The 
namespace must already
    +        |                              exist in the cluster. (Default: 
default).
    --- End diff --
    
    Actually k8s does not yet support `kill` and `status`, nor does it support 
`spark.cores.max` yet. Updated `validateKillArguments` and 
`validateStatusRequestArguments`.


---

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

Reply via email to