pmem / pmemstream

Other
9 stars 13 forks source link

Fix append benchmark for concurrent ops #244

Closed igchor closed 2 years ago

igchor commented 2 years ago

Current implementation only allows to append data concurrently to separate regions.

Also, add support for async appends.


This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #244 (494c543) into master (52282bc) will decrease coverage by 0.26%. The diff coverage is n/a.

:exclamation: Current head 494c543 differs from pull request most recent head f50e1cf. Consider uploading reports for the commit f50e1cf to get more accurate results

@@            Coverage Diff             @@
##           master     #244      +/-   ##
==========================================
- Coverage   89.48%   89.21%   -0.27%     
==========================================
  Files           9        9              
  Lines         742      742              
  Branches      134      134              
==========================================
- Hits          664      662       -2     
  Misses         47       47              
- Partials       31       33       +2     
Flag Coverage Δ
tests_gcc_debug_cpp17 89.21% <ø> (-0.27%) :arrow_down:

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

Impacted Files Coverage Δ
pmemstream/src/iterator.c 91.81% <0.00%> (-0.91%) :arrow_down:
pmemstream/src/libpmemstream.c 87.90% <0.00%> (-0.26%) :arrow_down:

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