Skip to content

[Backport 3.5] Leveraging segment-global ordinal mapping for efficient terms aggrega…#20717

Merged
jainankitk merged 3 commits into3.5from
backport/backport-20683-to-3.5
Feb 24, 2026
Merged

[Backport 3.5] Leveraging segment-global ordinal mapping for efficient terms aggrega…#20717
jainankitk merged 3 commits into3.5from
backport/backport-20683-to-3.5

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

@opensearch-trigger-bot opensearch-trigger-bot bot commented Feb 23, 2026

Backport 854f882 from #20683.

@github-actions
Leveraging segment-global ordinal mapping for efficient terms aggrega… (
ecfcc70 
#20683)

Signed-off-by: Ankit Jain <jainankitk@apache.org>
(cherry picked from commit 854f882)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@jainankitk
Update CHANGELOG.md to remove outdated entries
6674b3c 
Removed several entries related to fixes and updates in the CHANGELOG.

Signed-off-by: Ankit Jain <jainankitk@apache.org>
@jainankitk
Copy link
Copy Markdown
Contributor

jainankitk commented Feb 23, 2026

Need to merge #20718 before this. Working on that now

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 23, 2026

❌ Gradle check result for 6674b3c: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

❌ Gradle check result for abdf306: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@jainankitk
Copy link
Copy Markdown
Contributor

jainankitk commented Feb 24, 2026

Unrelated test failures, retrying gradle check:

[Test Result](https://build.ci.opensearch.org/job/gradle-check/71723/testReport/) (2 failures / +2)

    [org.opensearch.smoketest.SmokeTestMultiNodeClientYamlTestSuiteIT.test {yaml=search.aggregation/20_terms/numeric profiler}](https://build.ci.opensearch.org/job/gradle-check/71723/testReport/junit/org.opensearch.smoketest/SmokeTestMultiNodeClientYamlTestSuiteIT/test__yaml_search_aggregation_20_terms_numeric_profiler_/)
    [org.opensearch.action.admin.cluster.node.tasks.ResourceAwareTasksTests.testTaskResourceTrackingDuringTaskCancellation](https://build.ci.opensearch.org/job/gradle-check/71723/testReport/junit/org.opensearch.action.admin.cluster.node.tasks/ResourceAwareTasksTests/testTaskResourceTrackingDuringTaskCancellation/)

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 24, 2026
Open
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

❌ Gradle check result for abdf306: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

❌ Gradle check result for abdf306: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 24, 2026
Open
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

✅ Gradle check result for abdf306: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (3.5@6aed8ff). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...ket/terms/GlobalOrdinalsStringTermsAggregator.java 80.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files 
@@          Coverage Diff           @@
##             3.5   #20717   +/-   ##
======================================
  Coverage       ?   73.26%           
  Complexity     ?    71927           
======================================
  Files          ?     5782           
  Lines          ?   329070           
  Branches       ?    47447           
======================================
  Hits           ?   241081           
  Misses         ?    68630           
  Partials       ?    19359

☔ 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.

@jainankitk jainankitk merged commit cdab17c into 3.5 Feb 24, 2026
42 of 48 checks passed
@jainankitk jainankitk deleted the backport/backport-20683-to-3.5 branch February 24, 2026 18:13
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 24, 2026
Closed
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Mar 25, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jainankitk jainankitk jainankitk approved these changes

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz is a code owner

@andrross andrross Awaiting requested review from andrross andrross is a code owner

@ashking94 ashking94 Awaiting requested review from ashking94 ashking94 is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@cwperks cwperks Awaiting requested review from cwperks cwperks is a code owner

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@jed326 jed326 Awaiting requested review from jed326 jed326 is a code owner

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal is a code owner

@mch2 mch2 Awaiting requested review from mch2 mch2 is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh is a code owner

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@reta reta Awaiting requested review from reta reta is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 is a code owner

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jainankitk