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

Update resiliency docs #19303

Merged
merged 2 commits into from
Jul 8, 2016

Conversation

ywelsch
Copy link
Contributor

@ywelsch ywelsch commented Jul 7, 2016

Adds clarifications about Jepsen tests and new section on issues with versioning.

Add clarifications about Jepsen tests and new section on issues with versioning.
@ywelsch ywelsch added the >docs General docs changes label Jul 7, 2016
=== The _version field may not uniquely identify document content during a network partition (STATUS: ONGOING)

When a primary is partitioned away from the cluster there is a short period of time until it detects this. During that time it will continue
indexing writes locally, thereby updated document versions. When it tries replicating the operation, however, it will discover that it is
Copy link
Member

Choose a reason for hiding this comment

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

Typo: updated -> updating

Copy link
Contributor

Choose a reason for hiding this comment

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

indexing -> to index

Copy link
Contributor

Choose a reason for hiding this comment

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

replicating -> to replicate

@jasontedor
Copy link
Member

LGTM.


When a primary is partitioned away from the cluster there is a short period of time until it detects this. During that time it will continue
indexing writes locally, thereby updated document versions. When it tries replicating the operation, however, it will discover that it is
partitioned away. It won't acknowledge the write and wait until the partition is resolved to negotiate with the master on how to proceed.
Copy link
Contributor

Choose a reason for hiding this comment

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

is partitioned -> has been partitioned

Copy link
Contributor

Choose a reason for hiding this comment

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

write and wait -> write and will wait

@clintongormley
Copy link
Contributor

Minor grammar changes, but LGTM

@ywelsch ywelsch merged commit 7dff8fb into elastic:master Jul 8, 2016
@ywelsch
Copy link
Contributor Author

ywelsch commented Jul 8, 2016

Thanks @jasontedor @clintongormley.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants