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] Histogram field type support for ValueCount and Avg aggregations #56099

Merged
merged 2 commits into from
May 4, 2020

Conversation

csoulios
Copy link
Contributor

@csoulios csoulios commented May 4, 2020

Backports #55933 to 7.x

Implements value_count and avg aggregations over Histogram fields as discussed in #53285

  • value_count returns the sum of all counts array of the histograms
  • avg computes a weighted average of the values array of the histogram by multiplying each value with its associated element in the counts array

csoulios and others added 2 commits May 4, 2020 10:45
…stic#55933)

Implements value_count and avg aggregations over Histogram fields as discussed in elastic#53285

- value_count returns the sum of all counts array of the histograms
- avg computes a weighted average of the values array of the histogram by multiplying each value with its associated element in the counts array
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 4, 2020
@csoulios csoulios merged commit c65f828 into elastic:7.x May 4, 2020
@csoulios csoulios deleted the histo-value-count-avg-7.x branch May 4, 2020 10:23
@roncohen
Copy link

roncohen commented May 5, 2020

any chance of a backport to 7.8?

@roncohen
Copy link

roncohen commented May 5, 2020

wonderful to have this in by the way. Thank you so much for working on it

@tvernum
Copy link
Contributor

tvernum commented May 5, 2020

@roncohen The 7.x branch is the dev branch for 7.8

@roncohen
Copy link

roncohen commented May 5, 2020

excellent! ✨thanks @tvernum

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

Successfully merging this pull request may close these issues.

4 participants