zz-jason / leanstore

LeanStore is a larger-than-memory database, optimized for NVMe SSD and multi-core CPU, achieves performance close to in-memory systems without having to keep all data in memory.
MIT License
22 stars 2 forks source link

chore: tidy btree related code #124

Closed zz-jason closed 2 weeks ago

zz-jason commented 2 weeks ago

What's changed and how does it work?

tidy btree node and worker thread code, improve code readibility

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 60.90909% with 86 lines in your changes missing coverage. Please review.

Project coverage is 57.32%. Comparing base (553012d) to head (16da3b0). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/btree/core/BTreeGeneric.cpp 20.58% 24 Missing and 3 partials :warning:
src/btree/core/BTreeNode.cpp 59.37% 23 Missing and 3 partials :warning:
...clude/leanstore/btree/core/PessimisticIterator.hpp 64.00% 0 Missing and 9 partials :warning:
include/leanstore/btree/core/BTreeNode.hpp 85.71% 4 Missing and 3 partials :warning:
src/btree/BasicKV.cpp 12.50% 5 Missing and 2 partials :warning:
...nstore/btree/core/PessimisticExclusiveIterator.hpp 0.00% 1 Missing and 2 partials :warning:
src/concurrency/HistoryStorage.cpp 57.14% 2 Missing and 1 partial :warning:
include/leanstore/btree/core/BTreeGeneric.hpp 88.88% 0 Missing and 2 partials :warning:
src/btree/TransactionKV.cpp 66.66% 1 Missing :warning:
src/btree/Tuple.cpp 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #124 +/- ## ========================================== + Coverage 56.99% 57.32% +0.32% ========================================== Files 97 97 Lines 7044 7037 -7 Branches 3382 3165 -217 ========================================== + Hits 4015 4034 +19 + Misses 2657 2599 -58 - Partials 372 404 +32 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.