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] Update ES intro for stretched clusters #77651

Merged
merged 2 commits into from
Sep 13, 2021
Merged

[DOCS] Update ES intro for stretched clusters #77651

merged 2 commits into from
Sep 13, 2021

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Sep 13, 2021

PR #77360 clarifies that a cluster's nodes don't need to be in the same data
center. This adds a similar clarification to the ES introduction docs.

Preview

https://elasticsearch_77651.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/scalability.html#disaster-ccr

PR #77360 clarifies that a cluster's nodes don't need to be in the same data
center. This adds a similar clarification to the ES introduction docs.
@jrodewig jrodewig added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. v7.14.2 v7.15.1 v7.16.0 labels Sep 13, 2021
@jrodewig jrodewig marked this pull request as ready for review September 13, 2021 18:21
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Sep 13, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@jrodewig jrodewig added the >docs General docs changes label Sep 13, 2021
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Sep 13, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

Copy link
Contributor

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

LGTM although I left an optional suggestion.

Comment on lines 243 to 244
nearby data centers. Balancing shards across nodes that are far away or not well
connected simply takes too long. However, to maintain high availability, you
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the sentence about balancing shards doesn't really fit here. Basically any request might go between nodes and suffer from latency or disruption, and things like searches are likely a lot more latency-sensitive than shard movements. Maybe just drop it or maybe replace it with something along those lines instead.

Suggested change
nearby data centers. Balancing shards across nodes that are far away or not well
connected simply takes too long. However, to maintain high availability, you
nearby data centers. However, to maintain high availability, you

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great suggestion. Thanks, David!

@jrodewig jrodewig merged commit 88fcb67 into elastic:master Sep 13, 2021
@jrodewig jrodewig deleted the docs__fix-scalability branch September 13, 2021 20:50
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

jrodewig added a commit that referenced this pull request Sep 14, 2021
PR #77360 clarifies that a cluster's nodes don't need to be in the same data
center. This adds a similar clarification to the ES introduction docs.

Co-authored-by: David Turner <[email protected]>
jrodewig added a commit that referenced this pull request Sep 14, 2021
PR #77360 clarifies that a cluster's nodes don't need to be in the same data
center. This adds a similar clarification to the ES introduction docs.

Co-authored-by: David Turner <[email protected]>
jrodewig added a commit that referenced this pull request Sep 14, 2021
PR #77360 clarifies that a cluster's nodes don't need to be in the same data
center. This adds a similar clarification to the ES introduction docs.

Co-authored-by: David Turner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >docs General docs changes Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. Team:Docs Meta label for docs team v7.14.2 v7.15.1 v7.16.0 v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants