SmartRedis icon indicating copy to clipboard operation
SmartRedis copied to clipboard

Logging for SmartRedis

Open billschereriii opened this issue 2 years ago • 2 comments

billschereriii avatar Nov 16 '22 23:11 billschereriii

Initial C++ implementation is code complete, but not tested (or even compiled!). Provided for team review of implementation direction only

billschereriii avatar Nov 16 '22 23:11 billschereriii

Codecov Report

Merging #281 (3ffbe06) into develop (07f1fa1) will increase coverage by 14.81%. The diff coverage is 71.42%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           develop     #281       +/-   ##
============================================
+ Coverage    77.93%   92.74%   +14.81%     
============================================
  Files           57        9       -48     
  Lines         3666      703     -2963     
============================================
- Hits          2857      652     -2205     
+ Misses         809       51      -758     
Impacted Files Coverage Δ
src/python/module/smartredis/dataset.py 100.00% <ø> (ø)
src/python/module/smartredis/logcontext.py 52.38% <52.38%> (ø)
src/python/module/smartredis/srobject.py 52.38% <52.38%> (ø)
src/python/module/smartredis/__init__.py 100.00% <100.00%> (ø)
src/python/module/smartredis/client.py 95.11% <100.00%> (ø)
src/python/module/smartredis/logger.py 100.00% <100.00%> (ø)
include/client.h
include/dataset.h
include/dbnode.h
include/redisserver.h
... and 47 more

codecov-commenter avatar Nov 28 '22 21:11 codecov-commenter