Document Master URL format in high availability set up#17584
Closed
MirrorZ wants to merge 2 commits intoapache:masterfrom
Closed
Document Master URL format in high availability set up#17584MirrorZ wants to merge 2 commits intoapache:masterfrom
MirrorZ wants to merge 2 commits intoapache:masterfrom
Conversation
Add documentation for adding master url in multi host, port format for standalone cluster with high availability with zookeeper. Referring documentation [Standby Masters with ZooKeeper](http://spark.apache.org/docs/latest/spark-standalone.html#standby-masters-with-zookeeper)
srowen
reviewed
Apr 10, 2017
| <tr><td> <code>spark://HOST:PORT</code> </td><td> Connect to the given <a href="spark-standalone.html">Spark standalone | ||
| cluster</a> master. The port must be whichever one your master is configured to use, which is 7077 by default. | ||
| </td></tr> | ||
| <tr><td> <code>spark://HOST1:PORT1,HOST2:PORT2</code> </td><td> Connect to the given <a href="spark-standalone.html#standby-masters-with-zookeeper">Spark standalone |
Member
There was a problem hiding this comment.
zookeeper -> Zookeeper
Isn't this already documented then? I get that sometimes it's useful o doc something two places but I wonder if this is just redundant.
Contributor
Author
There was a problem hiding this comment.
I came across a situation where the developers using the spark driver believed that it was not possible to accept hosts in the mentioned format because it wasn't mentioned in the docs, so I found it useful to add the detail and submit a PR.
Cheers!
Update master url description for high availability master set up with Zookeeper, changing zookeeper -> Zookeeper.
srowen
approved these changes
Apr 10, 2017
|
Test build #3657 has finished for PR 17584 at commit
|
Member
|
merged to master |
peter-toth
pushed a commit
to peter-toth/spark
that referenced
this pull request
Oct 6, 2018
## What changes were proposed in this pull request? Add documentation for adding master url in multi host, port format for standalone cluster with high availability with zookeeper. Referring documentation [Standby Masters with ZooKeeper](http://spark.apache.org/docs/latest/spark-standalone.html#standby-masters-with-zookeeper) ## How was this patch tested? Documenting the functionality already present. Author: MirrorZ <chandrika3437@gmail.com> Closes apache#17584 from MirrorZ/master.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
Add documentation for adding master url in multi host, port format for standalone cluster with high availability with zookeeper.
Referring documentation Standby Masters with ZooKeeper
How was this patch tested?
Documenting the functionality already present.