huskysun commented on a change in pull request #32701:
URL: https://github.com/apache/spark/pull/32701#discussion_r642193027
##########
File path: docs/submitting-applications.md
##########
@@ -146,7 +146,7 @@ export HADOOP_CONF_DIR=XXX
./bin/spark-submit \
--class org.apache.spark.examples.SparkPi \
--master k8s://xx.yy.zz.ww:443 \
- --deploy-mode cluster \
+ --deploy-mode cluster \ # can be client for client mode
Review comment:
@yaooqinn @dongjoon-hyun Thanks for the review. Yeah I should've changed
L145 as well, from `# Run on a Kubernetes cluster in cluster deploy mode` to `#
Run on a Kubernetes cluster`. I was trying to mimicking L117. However I won't
do that anymore, because:
> you cannot add # after \ in bash
You're right, `#` can come after `\`. Then, L122 should also be fixed. I
will revert this line, and also fix L122. Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
