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

[7.x] Fix rate agg with custom _doc_count #79449

Merged
merged 2 commits into from
Oct 19, 2021

Conversation

csoulios
Copy link
Contributor

Backports #79346 to 7.x

When running a rate aggregation without setting the field parameter, the result is computed based on the bucket doc_count.

This PR adds support for a custom _doc_count field.

Closes #77734

csoulios and others added 2 commits October 19, 2021 13:27
When running a rate aggregation without setting the field parameter, the result is computed based on the bucket doc_count.

This PR adds support for a custom _doc_count field.

Closes elastic#77734
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 19, 2021
@elasticmachine
Copy link
Collaborator

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

@csoulios csoulios merged commit ffc61a2 into elastic:7.x Oct 19, 2021
@csoulios csoulios deleted the fix-rate-agg-7x branch October 19, 2021 11:48
@csoulios csoulios added v7.16.0 and removed v7.16.1 labels Oct 19, 2021
csoulios added a commit that referenced this pull request Oct 20, 2021
Now that the rate agg fix (#79346) was backported to v7.16 (#79449), we change the minimum version
for the test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants