Skip to content

TopN selection for streaming terms aggregations - #20481

Merged
rishabhmaurya merged 8 commits into
opensearch-project:mainfrom
rishabhmaurya:sAgg-topK
Jan 28, 2026
Merged

TopN selection for streaming terms aggregations#20481
rishabhmaurya merged 8 commits into
opensearch-project:mainfrom
rishabhmaurya:sAgg-topK

Addressed PR comments

c7ea430
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jan 28, 2026 in 0s

73.35% (target 70.00%)

View this Pull Request on Codecov

73.35% (target 70.00%)

Details

Codecov Report

❌ Patch coverage is 90.73171% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.35%. Comparing base (672039d) to head (c7ea430).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
...ons/bucket/terms/StreamNumericTermsAggregator.java 91.34% 5 Missing and 4 partials ⚠️
...ions/bucket/terms/StreamStringTermsAggregator.java 91.39% 5 Missing and 3 partials ⚠️
.../main/java/org/opensearch/index/IndexSettings.java 66.66% 2 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main   #20481      +/-   ##
============================================
+ Coverage     73.25%   73.35%   +0.10%     
- Complexity    71979    72092     +113     
============================================
  Files          5796     5796              
  Lines        329287   329539     +252     
  Branches      47419    47465      +46     
============================================
+ Hits         241203   241725     +522     
+ Misses        68759    68502     -257     
+ Partials      19325    19312      -13

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
View more details on Codecov