danielealbano commented 2 years ago

This PR implements a mechanism to flush the storage db synchronously.

It also includes some code of a previous PR to calculate the size of the DB as it wasn't initially included.

The mechanism is required by the redis command FLUSH [SYNC].

codecov[bot] commented 2 years ago

Codecov Report

Merging #188 (01c9b93) into main (f875831) will decrease coverage by 0.32%. The diff coverage is 3.45%.

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   79.27%   78.96%   -0.32%     
==========================================
  Files         106      107       +1     
  Lines        6866     6895      +29     
==========================================
+ Hits         5443     5444       +1     
- Misses       1423     1451      +28

Impacted Files	Coverage Δ
...a_structures/hashtable/mcmp/hashtable_op_get_key.c	`0.00% <0.00%> (ø)`
src/storage/db/storage_db.c	`56.73% <5.56%> (-1.95%)`	:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 01c9b936ae39bf2349b3f7a2fe01c75994d476f6 into f8758317d8d1eb762c09e1e9954b5cbfc375b4e7 - view on LGTM.com

new alerts:

1 for Missing return statement

danielealbano / cachegrand

Implement storage db flush sync #188

Codecov Report