-
Notifications
You must be signed in to change notification settings - Fork 525
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): hummock support ttl ability with compaction_filter #3438
Conversation
032c782
to
ccd14ad
Compare
9f66533
to
ec9c2d7
Compare
ccd14ad
to
c8de7ba
Compare
ec9c2d7
to
bee1f68
Compare
src/storage/src/hummock/compactor.rs
Outdated
match table_id { | ||
Some(table_id) => { | ||
let ttl = self.table_id_to_ttl[&table_id]; | ||
let epoch = get_epoch(key); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get epoch and table_id in once call for performance
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shall we need a new function to get epoch and table_id in once call ? (not exist now)
fn extract_table_id_and_epoch(full_key: &[u8]) -> (Option<u32>, Epoch)
in this case it means that we no need to get_epoch if table_id not exist
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. A new function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix it
3df6c68
to
5244321
Compare
Codecov Report
@@ Coverage Diff @@
## main #3438 +/- ##
==========================================
+ Coverage 74.37% 74.42% +0.04%
==========================================
Files 769 769
Lines 107047 107270 +223
==========================================
+ Hits 79621 79838 +217
- Misses 27426 27432 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
fix(storage): fix cargo.toml
5244321
to
c1719bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
hummock support ttl ability with compaction_filter
Checklist
Refer to a related PR or issue link (optional)