pmem / pmemstream

Other
9 stars 13 forks source link

[tests] Improve thread number calculation in concurrent iterate test #240

Closed KFilipek closed 2 years ago

KFilipek commented 2 years ago

Baseline execution: 27s After change: 21s

Commit message: This commit changes the way how the numbers of threads are drawn. Since now generated the number of synchronous concurrent appends is the difference between the number of async appends and max write concurrency.


This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #240 (6b9c068) into master (c5013bd) will decrease coverage by 0.13%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
- Coverage   88.98%   88.84%   -0.14%     
==========================================
  Files           9        9              
  Lines         726      726              
  Branches      127      127              
==========================================
- Hits          646      645       -1     
  Misses         48       48              
- Partials       32       33       +1     
Flag Coverage Δ
tests_gcc_debug_cpp17 88.84% <ø> (-0.14%) :arrow_down:

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

Impacted Files Coverage Δ
pmemstream/src/region.c 87.50% <0.00%> (-0.79%) :arrow_down:

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