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

Increase search.max_bucket default value by one #70645

Merged
merged 1 commit into from
Mar 23, 2021

Conversation

iverase
Copy link
Contributor

@iverase iverase commented Mar 22, 2021

In #57042, the default value for search.max_bucket was increased to 65,535. Unfortunately this value is short by one for some geo aggregations. For example if you want to aggregate on a geo tile with a scaling factor of 8, it means that you might be generating 256 * 256 buckets, which is one more than the default.

This PR increase the default by one (65,536).

@iverase iverase added >enhancement :Analytics/Geo Indexing, search aggregations of geo points and shapes :Analytics/Aggregations Aggregations v8.0.0 v7.13.0 labels Mar 22, 2021
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 22, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@nik9000
Copy link
Member

nik9000 commented Mar 22, 2021 via email

@iverase iverase merged commit b81bb42 into elastic:master Mar 23, 2021
@iverase iverase deleted the increaseMaxBucket branch March 23, 2021 08:24
jrodewig added a commit that referenced this pull request Jun 29, 2021
jrodewig added a commit that referenced this pull request Jun 29, 2021
jrodewig added a commit that referenced this pull request Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations :Analytics/Geo Indexing, search aggregations of geo points and shapes >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v7.13.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants