[DOCS] Add note that write indices are not replicated#82997
Merged
lockewritesdocs merged 3 commits intoelastic:masterfrom Jan 28, 2022
Merged
Conversation
Collaborator
|
Pinging @elastic/es-docs (Team:Docs) |
Collaborator
|
Pinging @elastic/es-distributed (Team:Distributed) |
Contributor
Author
|
@elasticmachine run elasticsearch-ci/docs |
bpintea
approved these changes
Jan 25, 2022
Contributor
bpintea
left a comment
There was a problem hiding this comment.
LGTM, but would have a member of @elastic/es-distributed review it too.
lockewritesdocs
commented
Jan 25, 2022
docs/reference/ccr/index.asciidoc
Outdated
|
|
||
| NOTE: Although changes to aliases on the leader index are replicated to follower | ||
| indices, write indices are ignored. Follower indices can't accept direct writes, | ||
| so any indices where `is_write_index` is `true` are not replicated. |
Contributor
Author
There was a problem hiding this comment.
so any indices where
is_write_indexistrueare not replicated.
@DaveCTurner, should this be "any indices" or "any index aliases"?
Contributor
There was a problem hiding this comment.
The indices are replicated, only the information on the alias about is_write_index is not, that flag is always forced to false on the follower.
Contributor
Author
There was a problem hiding this comment.
The indices are replicated, only the information on the alias about
is_write_indexis not, that flag is always forced to false on the follower.
Thanks for the explanation @henningandersen! I updated the note text in 9b5260f -- let me know if that's accurate.
Contributor
henningandersen
left a comment
There was a problem hiding this comment.
Thanks for addressing this @lockewritesdocs , I think this needs a slight change to be correct.
docs/reference/ccr/index.asciidoc
Outdated
|
|
||
| NOTE: Although changes to aliases on the leader index are replicated to follower | ||
| indices, write indices are ignored. Follower indices can't accept direct writes, | ||
| so any indices where `is_write_index` is `true` are not replicated. |
Contributor
There was a problem hiding this comment.
The indices are replicated, only the information on the alias about is_write_index is not, that flag is always forced to false on the follower.
henningandersen
approved these changes
Jan 28, 2022
Contributor
henningandersen
left a comment
There was a problem hiding this comment.
Made a suggestion, otherwise LGTM.
Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
jtibshirani
pushed a commit
to jtibshirani/elasticsearch
that referenced
this pull request
Jan 28, 2022
* [DOCS] Add note that write indices are not replicated * Clarify note about is_write_index * Clarify leader aliases with is_write_index Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
lockewritesdocs
pushed a commit
to lockewritesdocs/elasticsearch
that referenced
this pull request
Jan 28, 2022
* [DOCS] Add note that write indices are not replicated * Clarify note about is_write_index * Clarify leader aliases with is_write_index Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
lockewritesdocs
pushed a commit
to lockewritesdocs/elasticsearch
that referenced
this pull request
Jan 28, 2022
* [DOCS] Add note that write indices are not replicated * Clarify note about is_write_index * Clarify leader aliases with is_write_index Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
lockewritesdocs
pushed a commit
that referenced
this pull request
Jan 28, 2022
* [DOCS] Add note that write indices are not replicated * Clarify note about is_write_index * Clarify leader aliases with is_write_index Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
lockewritesdocs
pushed a commit
that referenced
this pull request
Jan 28, 2022
* [DOCS] Add note that write indices are not replicated * Clarify note about is_write_index * Clarify leader aliases with is_write_index Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com> Co-authored-by: Henning Andersen <33268011+henningandersen@users.noreply.github.com>
weizijun
added a commit
to weizijun/elasticsearch
that referenced
this pull request
Jan 31, 2022
* upstream/master: (100 commits) Avoid duplicate _type fields in v7 compat layer (elastic#83239) Bump bundled JDK to 17.0.2+8 (elastic#83243) [DOCS] Correct header syntax (elastic#83275) Add unit tests for indices.recovery.max_bytes_per_sec default values (elastic#83261) [DOCS] Add note that write indices are not replicated (elastic#82997) Add notes on indexing to kNN search guide (elastic#83188) Fix get-snapshot-api :docs:integTest (elastic#83273) FilterPathBasedFilter support match fieldname with dot (elastic#83178) Fix compilation issues in example-plugins (elastic#83258) fix ClusterStateListener javadoc (elastic#83246) Speed up Building Indices Lookup in Metadata (elastic#83241) Mute whole suite for elastic#82502 (elastic#83252) Make PeerFinder log messages happier (elastic#83222) [Docs] Add supported _terms_enum field types (elastic#83244) Add an aggregator for IPv4 and IPv6 subnets (elastic#82410) [CI] Fix 70_time_series/default sort yaml test failures (elastic#83217) Update test-failure Issue Template to include "needs:triage" label elastic#83226 Add an index->step cache to the PolicyStepsRegistry (elastic#82316) Improve support for joda datetime to java datetime transition in Painless (elastic#83099) Fix joda migration for week based methods in Painless (elastic#83232) ... # Conflicts: # x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/v2/TransportRollupAction.java
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
6 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.
Adds a note to the CCR docs that write indices are ignored when indices are replicated from a leader to a follower.
Preview link: https://elasticsearch_82997.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/xpack-ccr.html#ccr-update-leader-index
Closes #82679