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

(Doc+) Split API | add where need to have sufficient disk and how much #119784

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stefnestor
Copy link
Contributor

👋 howdy, ES Dev!

Will you kindly confirm for Split API doc

  1. Current doc states ... . My belief is that we're telling customers they need sufficient disk on each node hosting primary shards of the index which are going to be split out. AFAICT the doc does not currently otherwise help users judge which node will "handle the split process" which could randomly be master-only in which case this callout would feel invalid.

    The node handling the split process must have sufficient free disk space to accommodate a second copy of the existing index.

  2. If Elastic Cloud hard-link not supported (internal link) restriction is ongoing?

TLDR: Can we add expectations about where split api's created primary shards will allocate so users know where to ensure sufficient disk?

🙏 TIA!

👋 howdy, ES Dev! 

Will you kindly confirm for [Split API](https://www.elastic.co/guide/en/elasticsearch/reference/master/indices-split-index.html) doc
1. Current doc states ... . My belief is that we're telling customers they need sufficient disk on each node hosting primary shards of the index which are going to be split out. AFAICT the doc does not currently otherwise help users judge which node will "handle the split process" which could randomly be master-only in which case this callout would feel wrong.

    > The node handling the split process must have sufficient free disk space to accommodate a second copy of the existing index. 

2. If Elastic Cloud [hard-link not supported](https://support.elastic.dev/knowledge/view/d2cd7697) (internal link) restriction is ongoing?

🙏 TIA!
@stefnestor stefnestor added >docs General docs changes :Data Management/Indices APIs APIs to create and manage indices and templates Team:Data Management Meta label for data/management team labels Jan 8, 2025
Copy link
Contributor

github-actions bot commented Jan 8, 2025

Documentation preview:

@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Jan 8, 2025
@elasticsearchmachine
Copy link
Collaborator

@stefnestor please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticsearchmachine elasticsearchmachine added v9.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Jan 8, 2025
@lcawl
Copy link
Contributor

lcawl commented Jan 8, 2025

Hi! This is a drive-by comment that the page affected by this PR will be removed from the main branch soon. To avoid loss of this information, it should also be added here: https://github.com/elastic/elasticsearch-specification/blob/main/specification/indices/split/IndicesSplitRequest.ts

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 external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants