-
Notifications
You must be signed in to change notification settings - Fork 526
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
refactor(storage): simplify state table iter with async stream #3272
Conversation
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
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.
L😍TM
// (st1page): Maybe we will have a "ChunkIter" trait which returns a chunk each time, so the name | ||
// "RowTableIter" is reserved now | ||
pub struct CellBasedTableRowIter<S: StateStore> { | ||
/// [`CellBasedIter`] iterates on the cell-based table. |
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.
No need to link to itself? 🤪
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.
I'm afraid the clippy will complain about this. 😁
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Leave me some time to review this PR😊 |
Codecov Report
@@ Coverage Diff @@
## main #3272 +/- ##
==========================================
- Coverage 73.23% 73.22% -0.01%
==========================================
Files 747 748 +1
Lines 101789 101767 -22
==========================================
- Hits 74545 74523 -22
Misses 27244 27244
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 |
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.
Good Job!
Signed-off-by: Bugen Zhao i@bugenzhao.com
What's changed and what's your intention?
As title. The diff of
cell_based_table
is totally unreadable, please review the latest version. 🤣Checklist
./risedev check
(or alias,./risedev c
)Refer to a related PR or issue link (optional)