Skip to content

Add queryDSL sum and average aggregations for exponential histograms#136544

Merged
JonasKunz merged 9 commits intoelastic:mainfrom
JonasKunz:exp-histo-querydsl-sum-avg
Oct 20, 2025
Merged

Add queryDSL sum and average aggregations for exponential histograms#136544
JonasKunz merged 9 commits intoelastic:mainfrom
JonasKunz:exp-histo-querydsl-sum-avg

Conversation

@JonasKunz
Copy link
Copy Markdown
Contributor

Part of #135625 , similar to #136163.

Adds support for the sum and average aggregations on exponential histograms. The implementations are based on the corresponding aggregations of aggregate metric double.

@JonasKunz JonasKunz added the :Analytics/Aggregations Aggregations label Oct 14, 2025
@JonasKunz JonasKunz self-assigned this Oct 14, 2025
@elasticsearchmachine elasticsearchmachine added v9.3.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Oct 14, 2025
@JonasKunz JonasKunz changed the title Exp histo querydsl sum avg Add queryDSL sum and average aggregations for exponential histograms Oct 14, 2025
@JonasKunz JonasKunz marked this pull request as ready for review October 14, 2025 14:22
@JonasKunz JonasKunz requested review from felixbarny and martijnvg and removed request for felixbarny October 14, 2025 14:22
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 14, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

Copy link
Copy Markdown
Member

@felixbarny felixbarny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…um-avg

# Conflicts:
#	x-pack/plugin/mapper-exponential-histogram/src/main/java/org/elasticsearch/xpack/exponentialhistogram/ExponentialHistogramMapperPlugin.java
Copy link
Copy Markdown
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@JonasKunz JonasKunz enabled auto-merge (squash) October 20, 2025 10:49
@JonasKunz JonasKunz merged commit 9ce05e5 into elastic:main Oct 20, 2025
34 checks passed
@JonasKunz JonasKunz deleted the exp-histo-querydsl-sum-avg branch October 20, 2025 11:59
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Oct 24, 2025
fzowl pushed a commit to voyage-ai/elasticsearch that referenced this pull request Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants