Skip to content

create-index: Default number of shards does not match v7 changes #30746

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

Closed
atc0005 opened this issue May 20, 2018 · 6 comments
Closed

create-index: Default number of shards does not match v7 changes #30746

atc0005 opened this issue May 20, 2018 · 6 comments
Assignees
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates

Comments

@atc0005
Copy link
Contributor

atc0005 commented May 20, 2018

I saw in the blog post that #30539 changes the default number of primary shards to 1. Looking over the docs I found that in master the create-index API coverage still showed 5 as the default number instead of 1. I assume that this was just missed in the recent round of doc updates from 4852f34.

@jasontedor
Copy link
Member

Thanks for noticing this. Do you want to submit a pull request otherwise I’ll push a fix for this on Monday?

@jasontedor jasontedor self-assigned this May 20, 2018
@jasontedor jasontedor added the :Data Management/Indices APIs APIs to create and manage indices and templates label May 20, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jasontedor
Copy link
Member

Oh I see we were racing and you already submitted a pull request. Thanks for that!

@atc0005
Copy link
Contributor Author

atc0005 commented May 20, 2018

Oh I see we were racing and you already submitted a pull request. Thanks for that!

You're welcome! I started with the PR, but held back once I noticed that one of the "make sure you do X" checklist items included creating an issue first.

Please let me know if I missed anything or need to go about the change in a different way. I'm new to ES, so I may have missed something obvious to long-time users.

@jasontedor
Copy link
Member

Please let me know if I missed anything or need to go about the change in a different way. I'm new to ES, so I may have missed something obvious to long-time users.

It’s perfect. Thanks for your interest and your contribution.

@atc0005
Copy link
Contributor Author

atc0005 commented May 20, 2018

It’s perfect. Thanks for your interest and your contribution.

Awesome! You're welcome, and thanks for the feedback.

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
Projects
None yet
Development

No branches or pull requests

3 participants