Skip to content

Add aggregation support#10

Closed
obukhow wants to merge 1 commit intosdqri:masterfrom
obukhow:agg
Closed

Add aggregation support#10
obukhow wants to merge 1 commit intosdqri:masterfrom
obukhow:agg

Conversation

@obukhow
Copy link
Copy Markdown

@obukhow obukhow commented Mar 30, 2025

Add aggregation support for queries

An aggregation summarizes your data as metrics, statistics, or other analytics.
https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations.html

Description

Started by implementing two most common used aggregations – terms bucket and stats metric aggregation.
Other aggregations type could be implemented by extending AggregationInterface

  • Bug fix
  • Feature addition
  • Code refactoring
  • Documentation update
  • Other (please explain):

Related Issue

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • I have made corresponding changes to the documentation.
  • I have added tests that ensure my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines Coverage Δ
effdsl.go 69.30% <100.00%> (+3.00%) ⬆️
search_aggregation.go 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sdqri sdqri mentioned this pull request Jul 16, 2025
@sdqri
Copy link
Copy Markdown
Owner

sdqri commented Feb 12, 2026

Thanks for the contribution. Aggregation support has since been implemented across metrics, bucket, and pipeline aggregations in v2.7.0, along with updated docs and tests. I’m closing this PR as it’s now superseded by the merged changes.

@sdqri sdqri closed this Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants