Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: Document the most important changes to zen discovery. #7746

Closed
wants to merge 3 commits into from

Conversation

martijnvg
Copy link
Member

No description provided.


Each cluster must have an elected master node in order to be fully operational. Once a node loses its elected master
node it will reject operations.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add some or all? i.e., "node it will reject some or all operations".

@bleskes
Copy link
Contributor

bleskes commented Sep 24, 2014

LGTM in general, left some minor comments. Would love @clintongormley or @polyfractal to have a look as well.

@martijnvg
Copy link
Member Author

Updated the PR and applied @bleskes's feedback.

what operations should be rejected when there is no active master.

The `discovery.zen.no_master_block` setting has two valid options:
* `all` - All operations on the node. I.e., both read & writes will be rejected.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this include read-only cluster-metadata APIs like Node Stats, Cluster State, etc? Should probably add a note regardless of what the answer is, just to be explicit.

@polyfractal
Copy link
Contributor

Left a few comments. LGTM overall! :)

@martijnvg
Copy link
Member Author

Updated the PR and applied @polyfractal's feedback.

@polyfractal
Copy link
Contributor

👍 LGTM

@martijnvg martijnvg closed this in fca4064 Sep 30, 2014
@jpountz jpountz removed the review label Oct 21, 2014
@martijnvg martijnvg deleted the docs/zen_changes branch May 18, 2015 23:30
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants