Skip to content

[Backport 2.17] [Backport 2.x] [Star Tree] Scaled Float Support#15716

Merged
gbbafna merged 1 commit intoopensearch-project:2.17from
sarthakaggarwal97:backport/backport-15660-to-2.17-lat-1
Sep 5, 2024
Merged

[Backport 2.17] [Backport 2.x] [Star Tree] Scaled Float Support#15716
gbbafna merged 1 commit intoopensearch-project:2.17from
sarthakaggarwal97:backport/backport-15660-to-2.17-lat-1

Conversation

@sarthakaggarwal97
Copy link
Copy Markdown
Contributor

@sarthakaggarwal97 sarthakaggarwal97 commented Sep 5, 2024

Description

Backport c801270 from #15660.

Related Issues

Resolves [Backport 2.x] [Star tree] Star tree index validations (#15533) (#15610)

…ch-project#15660)

---------

(cherry picked from commit 23f5c2f)

Signed-off-by: Sarthak Aggarwal <sarthagg@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit c801270)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 5, 2024

✅ Gradle check result for 047a9f1: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 5, 2024

Codecov Report

Attention: Patch coverage is 77.77778% with 20 lines in your changes missing coverage. Please review.

Project coverage is 71.65%. Comparing base (cea8eea) to head (047a9f1).
Report is 3 commits behind head on 2.17.

Files with missing lines Patch % Lines
...startree/builder/AbstractDocumentsFileManager.java 79.16% 2 Missing and 3 partials ⚠️
...datacube/startree/builder/BaseStarTreeBuilder.java 50.00% 1 Missing and 4 partials ⚠️
...ndex/compositeindex/datacube/DimensionFactory.java 20.00% 0 Missing and 4 partials ⚠️
...pensearch/index/mapper/ScaledFloatFieldMapper.java 60.00% 2 Missing ⚠️
...org/opensearch/index/mapper/NumberFieldMapper.java 87.50% 1 Missing and 1 partial ⚠️
...ube/startree/aggregators/MetricAggregatorInfo.java 75.00% 1 Missing ⚠️
...g/opensearch/index/mapper/DocCountFieldMapper.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               2.17   #15716      +/-   ##
============================================
- Coverage     71.68%   71.65%   -0.03%     
+ Complexity    64322    64315       -7     
============================================
  Files          5262     5262              
  Lines        300929   300921       -8     
  Branches      43803    43809       +6     
============================================
- Hits         215709   215636      -73     
- Misses        67105    67200      +95     
+ Partials      18115    18085      -30     

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

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