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: add redis caching #179

Closed s0nicboOm closed 1 year ago

s0nicboOm commented 1 year ago

PR introduces:

  1. Adding caching for redis registry (includes code rearrangement)
  2. Fix mlflow test cases
codecov[bot] commented 1 year ago

Codecov Report

Merging #179 (05f4463) into main (0f7e6e0) will decrease coverage by 0.08%. The diff coverage is 95.83%.

@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
- Coverage   96.80%   96.73%   -0.08%     
==========================================
  Files          36       36              
  Lines        1503     1530      +27     
  Branches      113      117       +4     
==========================================
+ Hits         1455     1480      +25     
  Misses         36       36              
- Partials       12       14       +2     
Impacted Files Coverage Δ
numalogic/registry/redis_registry.py 97.61% <95.45%> (-1.42%) :arrow_down:
numalogic/registry/artifact.py 91.66% <100.00%> (+0.36%) :arrow_up:
numalogic/registry/localcache.py 94.44% <100.00%> (+0.69%) :arrow_up:

... and 2 files with indirect coverage changes

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