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 a soft limit on the number of shards that can be queried in a single search request. #17396

Merged

Conversation

jpountz
Copy link
Contributor

@jpountz jpountz commented Mar 30, 2016

This commit adds the new action.search.shard_count.limit setting which
configures the maximum number of shards that can be queried in a single search
request. It has a default value of 1000.

@jpountz jpountz added >enhancement review :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1 labels Mar 30, 2016
@jpountz
Copy link
Contributor Author

jpountz commented Mar 30, 2016

The goal was initially to not apply this limit to count requests, but it is not trivial to check and after some discussions with @clintongormley, this is not so much of an issue since the limit is easy to change and you can get counts of docs in indices using eg. _cat/count which has no limit (and does not need one).

@jimczi
Copy link
Contributor

jimczi commented Mar 30, 2016

@jpountz thanks for the explanation.
LGTM

…gle search request. elastic#17396

This commit adds the new `action.search.shard_count.limit` setting which
configures the maximum number of shards that can be queried in a single search
request. It has a default value of 1000.
@jpountz jpountz force-pushed the feature/search_shard_count_soft_limit branch from b7e270c to d7179ca Compare March 30, 2016 14:55
@jpountz jpountz merged commit d7179ca into elastic:master Mar 30, 2016
@jpountz jpountz deleted the feature/search_shard_count_soft_limit branch March 30, 2016 15:00
@jpountz
Copy link
Contributor Author

jpountz commented Mar 30, 2016

Thanks @jimferenczi !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants