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

WIP kvs-watch: do not load all KVS blobs at once #6532

Closed

kvs-watch: do not load all KVS blobs at once

8604a3d
Select commit
Loading
Failed to load commit list.
Closed

WIP kvs-watch: do not load all KVS blobs at once #6532

kvs-watch: do not load all KVS blobs at once
8604a3d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Dec 21, 2024 in 0s

83.65% (+0.02%) compared to 9278fa8

View this Pull Request on Codecov

83.65% (+0.02%) compared to 9278fa8

Details

Codecov Report

Attention: Patch coverage is 83.33333% with 8 lines in your changes missing coverage. Please review.

Project coverage is 83.65%. Comparing base (9278fa8) to head (8604a3d).
Report is 11 commits behind head on master.

Files with missing lines Patch % Lines
src/modules/kvs-watch/kvs-watch.c 83.33% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6532      +/-   ##
==========================================
+ Coverage   83.62%   83.65%   +0.02%     
==========================================
  Files         522      522              
  Lines       87809    87829      +20     
==========================================
+ Hits        73433    73473      +40     
+ Misses      14376    14356      -20     
Files with missing lines Coverage Δ
src/modules/kvs-watch/kvs-watch.c 77.93% <83.33%> (-0.38%) ⬇️

... and 8 files with indirect coverage changes