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

fix(repo-server): excess git request when using multi-source and ref sources (Issue #14725) #16309

Closed
wants to merge 5 commits into from

feat: move to two level cache, reconfigure cache interface to provide…

33808ec
Select commit
Loading
Failed to load commit list.
Closed

fix(repo-server): excess git request when using multi-source and ref sources (Issue #14725) #16309

feat: move to two level cache, reconfigure cache interface to provide…
33808ec
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Nov 19, 2023 in 0s

49.86% (+0.31%) compared to 6fa9b17

View this Pull Request on Codecov

49.86% (+0.31%) compared to 6fa9b17

Details

Codecov Report

Attention: 39 lines in your changes are missing coverage. Please review.

Comparison is base (6fa9b17) 49.55% compared to head (33808ec) 49.86%.

Files Patch % Lines
util/git/client.go 11.76% 15 Missing ⚠️
util/cache/cache.go 70.73% 9 Missing and 3 partials ⚠️
util/cache/twolevelclient.go 55.00% 9 Missing ⚠️
reposerver/cache/cache.go 97.81% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #16309      +/-   ##
==========================================
+ Coverage   49.55%   49.86%   +0.31%     
==========================================
  Files         269      269              
  Lines       47039    47215     +176     
==========================================
+ Hits        23309    23545     +236     
+ Misses      21444    21373      -71     
- Partials     2286     2297      +11     

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