Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] [Star tree] Moving compositeCodec to composite912Codec and dropping composite99Codec #16346

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport b5917c5 from #16227.

…omposite99Codec (#16227)

* Refactoring builder tests
* composite912 codec refactor changes

---------

Signed-off-by: Bharathwaj G <[email protected]>
(cherry picked from commit b5917c5)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

❕ Gradle check result for 54edcbf: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Copy link

codecov bot commented Oct 16, 2024

Codecov Report

Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.82%. Comparing base (b68fff8) to head (54edcbf).
Report is 5 commits behind head on 2.x.

Files with missing lines Patch % Lines
...site/composite912/Composite912DocValuesWriter.java 83.33% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #16346      +/-   ##
============================================
+ Coverage     71.75%   71.82%   +0.07%     
- Complexity    65036    65104      +68     
============================================
  Files          5305     5304       -1     
  Lines        304043   304050       +7     
  Branches      44221    44219       -2     
============================================
+ Hits         218176   218398     +222     
+ Misses        67717    67494     -223     
- Partials      18150    18158       +8     

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

@reta reta merged commit 9ee936b into 2.x Oct 16, 2024
64 of 65 checks passed
@github-actions github-actions bot deleted the backport/backport-16227-to-2.x branch October 16, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants