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

    https://github.com/apache/spark/pull/19717#discussion_r154503012
  
    --- 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
    +        |                              application must be launched. The 
namespace must already
    +        |                              exist in the cluster. (Default: 
default).
    --- End diff --
    
    now that docker image is without a default value 
(`spark.kubernetes.*.docker.image`), (yes that discussion is ongoing) I wonder 
if it makes sense to bubble that up as a `--param` for visibility/convenient.
    
    tbh I'm generally against adding `--param` to submit because of the 
potential confusion it can cause, but since we are here and there's a 
`--kubernetes-namespace`


---

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

Reply via email to