Skip to content

[Backport 3.4] Aggregation Array OOB Error #20204#20219

Merged
prudhvigodithi merged 1 commit intoopensearch-project:3.4from
sandeshkr419:agg
Dec 12, 2025
Merged

[Backport 3.4] Aggregation Array OOB Error #20204#20219
prudhvigodithi merged 1 commit intoopensearch-project:3.4from
sandeshkr419:agg

Conversation

@sandeshkr419
Copy link
Member

@sandeshkr419 sandeshkr419 commented Dec 12, 2025

Description

Manual backport to 3.4: #20204

Related Issues

Resolves #19365

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Co-authored-by: Sandesh Kumar <sandeshkr419@gmail.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 12, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

@sandeshkr419 sandeshkr419 added backport PRs or issues specific to backporting features or enhancments skip-changelog bug Something isn't working Search:Aggregations labels Dec 12, 2025
@github-actions
Copy link
Contributor

❌ Gradle check result for 4c02f10: 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
Contributor

✅ Gradle check result for 4c02f10: SUCCESS

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.22%. Comparing base (7d87ca2) to head (4c02f10).
⚠️ Report is 2 commits behind head on 3.4.

Additional details and impacted files
@@             Coverage Diff              @@
##                3.4   #20219      +/-   ##
============================================
- Coverage     73.24%   73.22%   -0.02%     
- Complexity    71737    71745       +8     
============================================
  Files          5794     5794              
  Lines        328218   328218              
  Branches      47261    47261              
============================================
- Hits         240404   240347      -57     
- Misses        68504    68617     +113     
+ Partials      19310    19254      -56     

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

@prudhvigodithi prudhvigodithi merged commit 62c3bdc into opensearch-project:3.4 Dec 12, 2025
71 of 74 checks passed
@sandeshkr419 sandeshkr419 deleted the agg branch December 12, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport PRs or issues specific to backporting features or enhancments bug Something isn't working Search:Aggregations skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants