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

Abstract AsyncShardFetch cache to allow restructuring for other caching strategies #12441

Merged
merged 14 commits into from
Mar 11, 2024

Merge remote-tracking branch 'origin/main' into abstract-cache

d7903f7
Select commit
Loading
Failed to load commit list.
Merged

Abstract AsyncShardFetch cache to allow restructuring for other caching strategies #12441

Merge remote-tracking branch 'origin/main' into abstract-cache
d7903f7
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 11, 2024 in 0s

71.35% (target 70.00%)

View this Pull Request on Codecov

71.35% (target 70.00%)

Details

Codecov Report

Attention: Patch coverage is 81.33333% with 28 lines in your changes are missing coverage. Please review.

Project coverage is 71.35%. Comparing base (2b17902) to head (d7903f7).

Files Patch % Lines
...a/org/opensearch/gateway/AsyncShardFetchCache.java 78.33% 10 Missing and 16 partials ⚠️
...n/java/org/opensearch/gateway/AsyncShardFetch.java 93.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #12441      +/-   ##
============================================
- Coverage     71.43%   71.35%   -0.08%     
- Complexity    59960    59982      +22     
============================================
  Files          4984     4985       +1     
  Lines        282247   282275      +28     
  Branches      40952    40946       -6     
============================================
- Hits         201617   201415     -202     
- Misses        63954    64198     +244     
+ Partials      16676    16662      -14     

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