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

feat(storage)!: reduce remove_blocks IO #472

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Rexagon
Copy link
Member

@Rexagon Rexagon commented Nov 24, 2024

BREAKING: A new column family is required now - full_block_ids

@Rexagon Rexagon force-pushed the feature/reduce-remove-blocks-io branch 2 times, most recently from 2e8322a to 7936c43 Compare November 24, 2024 13:50
BREAKING: A new column family is required now - `full_block_ids`
@Rexagon Rexagon force-pushed the feature/reduce-remove-blocks-io branch from 7936c43 to 8ff4180 Compare November 24, 2024 13:51
Copy link

codecov bot commented Nov 24, 2024

Codecov Report

Attention: Patch coverage is 77.17391% with 21 lines in your changes missing coverage. Please review.

Project coverage is 42.99%. Comparing base (4755326) to head (8ff4180).

Files with missing lines Patch % Lines
storage/src/store/block/package_entry.rs 33.33% 18 Missing ⚠️
storage/src/store/block_handle/mod.rs 92.68% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #472      +/-   ##
==========================================
- Coverage   43.00%   42.99%   -0.02%     
==========================================
  Files         266      266              
  Lines       46484    46517      +33     
  Branches    46484    46517      +33     
==========================================
+ Hits        19989    19998       +9     
- Misses      25413    25440      +27     
+ Partials     1082     1079       -3     

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


🚨 Try these New Features:

}
}

impl ColumnFamilyOptions<Caches> for FullBlockIds {}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

set caches via default_block_based_table_factory

@0xdeafbeef
Copy link
Member

image
image
it's a huge win

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.

2 participants