-
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(hummock): introduce aligned allocator and cache file skeleton #3250
Conversation
020dcac
to
16ee3a6
Compare
Codecov Report
@@ Coverage Diff @@
## main #3250 +/- ##
==========================================
+ Coverage 73.22% 73.24% +0.02%
==========================================
Files 748 753 +5
Lines 101994 102280 +286
==========================================
+ Hits 74686 74920 +234
- Misses 27308 27360 +52
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 |
Maybe this PR is too immature, but I want to keep each PR small and easy to review, QwQ. |
LGTM for the allocator part. I haven't enough background knowledge to review the PR ( |
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.
Generally LGTM. Since it's not completed yet, we may improve later.
What's changed and what's your intention?
This is a one of the PRs for the file cache system, to avoid sending a huge PR all at once.
AlignedAllocator
for direct I/O buffer allocation.Checklist
./risedev check
(or alias,./risedev c
)Refer to a related PR or issue link (optional)
#198