Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/19717#discussion_r154567885
--- 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 --
I think if we eventually decide to not have default docker images, we
should make the options `--param` ones. I'm not sure if we want to make a call
and do that in this PR though. Can we defer this to a later time when we are
clearer on how we publish and maintain the images?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
