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

Add note on the danger of closed indices #61332

Merged

Conversation

DaveCTurner
Copy link
Contributor

Prior to #33888 it is dangerous to keep closed indices in your cluster
long-term: Elasticsearch does not maintain their shard copies so they
tend to get lost as the cluster migrates to new nodes. This risk isn't
documented today. This commit addresses that gap.

Prior to elastic#33888 it is dangerous to keep closed indices in your cluster
long-term: Elasticsearch does not maintain their shard copies so they
tend to get lost as the cluster migrates to new nodes. This risk isn't
documented today. This commit addresses that gap.
@DaveCTurner DaveCTurner added >docs General docs changes :Data Management/Indices APIs APIs to create and manage indices and templates v6.8.13 labels Aug 19, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Indices APIs)

@elasticmachine elasticmachine added Team:Docs Meta label for docs team Team:Data Management Meta label for data/management team labels Aug 19, 2020
Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

LGTM overall.

I left a few minor comments, but this is largely good as-is.

docs/reference/indices/open-close.asciidoc Outdated Show resolved Hide resolved
docs/reference/indices/open-close.asciidoc Outdated Show resolved Hide resolved
docs/reference/indices/open-close.asciidoc Outdated Show resolved Hide resolved
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
@DaveCTurner
Copy link
Contributor Author

Thanks James, I accepted your rewrite :)

https://www.elastic.co/cloud/elasticsearch-service/signup[try out the hosted service] for free.
Elasticsearch can be run on your own hardware or using our hosted Elasticsearch
Service on https://www.elastic.co/cloud[Elastic Cloud], which is available on
AWS, GCP and Azure. You can
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unrelated except that I was looking for other references to Cloud: reformat & mention Azure.

Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

Left a few minor suggestions. The attributes should probably be added,
but otherwise feel free to take/leave as you see fit.

docs/reference/indices/open-close.asciidoc Outdated Show resolved Hide resolved
docs/reference/setup/install.asciidoc Outdated Show resolved Hide resolved
DaveCTurner and others added 3 commits August 19, 2020 19:21
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner merged commit 37825c3 into elastic:6.8 Aug 20, 2020
@DaveCTurner DaveCTurner deleted the 2020-08-19-closed-index-danger-docs branch August 20, 2020 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates >docs General docs changes Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v6.8.13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants