Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: tidy btree related code #124

Merged
merged 1 commit into from
Aug 30, 2024
Merged

chore: tidy btree related code #124

merged 1 commit into from
Aug 30, 2024

Conversation

zz-jason
Copy link
Owner

What's changed and how does it work?

tidy btree node and worker thread code, improve code readibility

@zz-jason zz-jason force-pushed the jian.z/tidy-btree-node branch from 0a8e7cd to 5d10f26 Compare August 27, 2024 13:24
Copy link

codecov bot commented Aug 27, 2024

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 ⚠️
src/btree/core/BTreeNode.cpp 59.37% 23 Missing and 3 partials ⚠️
...clude/leanstore/btree/core/PessimisticIterator.hpp 64.00% 0 Missing and 9 partials ⚠️
include/leanstore/btree/core/BTreeNode.hpp 85.71% 4 Missing and 3 partials ⚠️
src/btree/BasicKV.cpp 12.50% 5 Missing and 2 partials ⚠️
...nstore/btree/core/PessimisticExclusiveIterator.hpp 0.00% 1 Missing and 2 partials ⚠️
src/concurrency/HistoryStorage.cpp 57.14% 2 Missing and 1 partial ⚠️
include/leanstore/btree/core/BTreeGeneric.hpp 88.88% 0 Missing and 2 partials ⚠️
src/btree/TransactionKV.cpp 66.66% 1 Missing ⚠️
src/btree/Tuple.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            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     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zz-jason zz-jason force-pushed the jian.z/tidy-btree-node branch from 5d10f26 to 0559c8c Compare August 30, 2024 00:15
@zz-jason zz-jason force-pushed the jian.z/tidy-btree-node branch from 0559c8c to 16da3b0 Compare August 30, 2024 00:39
@zz-jason zz-jason merged commit 5b23cfd into main Aug 30, 2024
5 checks passed
@zz-jason zz-jason deleted the jian.z/tidy-btree-node branch August 30, 2024 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant