numaproj / numalogic

Collection of operational time series ML models and tools
https://numalogic.numaproj.io/
Apache License 2.0
166 stars 28 forks source link

fix (RedisRegistry): avoid overwriting cache with the same key during load #223

Closed ab93 closed 1 year ago

ab93 commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Merging #223 (de953f9) into main (0b0daed) will decrease coverage by 0.14%. The diff coverage is 100.00%.

:exclamation: Current head de953f9 differs from pull request most recent head 6c3ae5f. Consider uploading reports for the commit 6c3ae5f to get more accurate results

@@            Coverage Diff             @@
##             main     #223      +/-   ##
==========================================
- Coverage   96.29%   96.15%   -0.14%     
==========================================
  Files          46       46              
  Lines        1942     1953      +11     
  Branches      155      157       +2     
==========================================
+ Hits         1870     1878       +8     
- Misses         52       54       +2     
- Partials       20       21       +1     
Impacted Files Coverage Δ
numalogic/registry/artifact.py 92.00% <100.00%> (+0.33%) :arrow_up:
numalogic/registry/localcache.py 96.00% <100.00%> (+1.55%) :arrow_up:
numalogic/registry/redis_registry.py 97.67% <100.00%> (+0.03%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more