You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "nchammas (via GitHub)" <gi...@apache.org> on 2026/07/25 18:51:25 UTC
[PR] [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs [spark]
nchammas opened a new pull request, #57536:
URL: https://github.com/apache/spark/pull/57536
### What changes were proposed in this pull request?
Miscellaneous typo fixes to the standalone and YARN deployment docs.
Add references to the standalone docs that are missing, like for Kubernetes and various `sbin` scripts.
### Why are the changes needed?
Just to improve the flow of these particular docs and add handy references a reader would expect to find.
### Does this PR introduce _any_ user-facing change?
Documentation only.
### How was this patch tested?
I built the docs using `SKIP_API=1 bundle exec jekyll serve` and reviewed them in my browser.
### Was this patch authored or co-authored using generative AI tooling?
GitHub Copilot helped find some of the typos.
--
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs [spark]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 commented on PR #57536:
URL: https://github.com/apache/spark/pull/57536#issuecomment-5205268901
**Merge Summary:**
- merged into master https://github.com/apache/spark/commit/80374dc2a548d92b123018ba6b3a506e434a59f2
- merged into branch-4.x https://github.com/apache/spark/commit/afa2cea7f5177c09ff6194145b67606b8ce60a03
*Posted by `merge_spark_pr.py`*
--
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs [spark]
Posted by "uros-b (via GitHub)" <gi...@apache.org>.
uros-b commented on PR #57536:
URL: https://github.com/apache/spark/pull/57536#issuecomment-5083079660
Thank you @nchammas!
--
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs [spark]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 closed pull request #57536: [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs
URL: https://github.com/apache/spark/pull/57536
--
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [MINOR][DOCS] Fix typos and add missing references to standalone and YARN deployment docs [spark]
Posted by "nchammas (via GitHub)" <gi...@apache.org>.
nchammas commented on code in PR #57536:
URL: https://github.com/apache/spark/pull/57536#discussion_r3652585361
##########
docs/spark-standalone.md:
##########
@@ -100,13 +100,16 @@ Once you've set up this file, you can launch or stop your cluster with the follo
- `sbin/start-master.sh` - Starts a master instance on the machine the script is executed on.
- `sbin/start-workers.sh` - Starts a worker instance on each machine specified in the `conf/workers` file.
- `sbin/start-worker.sh` - Starts a worker instance on the machine the script is executed on.
-- `sbin/start-connect-server.sh` - Starts a Spark Connect server on the machine the script is executed on.
+- `sbin/start-connect-server.sh` - Starts a [Spark Connect](spark-connect-overview.html) server on the machine the script is executed on.
+- `sbin/start-history-server.sh` - Starts the History Server, which lets you view logs for completed applications. Requires event logging to be enabled (see [Monitoring and Instrumentation](monitoring.html)).
Review Comment:
The section this is in is called "Cluster Launch Scripts", so I don't know if we want to leave mention of the History server out of here since it gets a dedicated mention in the monitoring page. The Thrift server also gets its own call out in the "Distributed SQL Engine" page (which is currently not in this list).
I like the idea of having a complete list of the `sbin` scripts users may be interested in, even if some of them get additional explanation elsewhere. In that case, I'd also add the Thrift server here.
What do you think, @uros-b?
--
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.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org