search

pmem / pmemstream

Other
9 stars 13 forks source link

Add shadow value to pmemstream_persisted_timestamp #249

Closed karczex closed 2 years ago

karczex commented 2 years ago

This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #249 (d0ab2bd) into master (52282bc) will decrease coverage by 0.19%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #249      +/-   ##
==========================================
- Coverage   89.48%   89.29%   -0.20%     
==========================================
  Files           9        9              
  Lines         742      747       +5     
  Branches      134      135       +1     
==========================================
+ Hits          664      667       +3     
- Misses         47       48       +1     
- Partials       31       32       +1
Flag Coverage Δ
tests_gcc_debug_cpp17 89.29% <ø> (-0.20%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pmemstream/src/libpmemstream.c 87.81% <0.00%> (-0.36%) :arrow_down:
pmemstream/src/libpmemstream_internal.h 100.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

igchor commented 2 years ago

Ref: https://pmem.io/blog/2021/09/concurrency-considerations-in-libpmemobj-cpp/