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

feat: introduce valid check into iterator #110

Closed zz-jason closed 1 month ago

zz-jason commented 1 month ago

What's changed and how does it work?

Ref to: https://github.com/zz-jason/leanstore/issues/97

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 69.51220% with 50 lines in your changes missing coverage. Please review.

Project coverage is 61.71%. Comparing base (2cc6019) to head (71e31c7).

Files Patch % Lines
...clude/leanstore/btree/core/PessimisticIterator.hpp 74.54% 12 Missing and 2 partials :warning:
src/btree/TransactionKV.cpp 48.14% 9 Missing and 5 partials :warning:
src/btree/BasicKV.cpp 0.00% 9 Missing and 1 partial :warning:
src/leanstore-c.cpp 57.14% 6 Missing :warning:
src/concurrency/HistoryStorage.cpp 25.00% 0 Missing and 3 partials :warning:
...nstore/btree/core/PessimisticExclusiveIterator.hpp 33.33% 2 Missing :warning:
tests/btree/BasicKvIteratorTest.cpp 98.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #110 +/- ## ========================================== + Coverage 59.89% 61.71% +1.81% ========================================== Files 118 119 +1 Lines 8756 8817 +61 Branches 3294 3469 +175 ========================================== + Hits 5244 5441 +197 + Misses 3129 3004 -125 + Partials 383 372 -11 ```

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