-
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
refactor(source): Move field datagen to common #3491
Conversation
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.
license-eye has totally checked 879 files.
Valid | Invalid | Ignored | Fixed |
---|---|---|---|
877 | 1 | 1 | 0 |
Click to see the invalid file list
- src/batch/benches/filter.rs
08081e9
to
809d740
Compare
Codecov Report
@@ Coverage Diff @@
## main #3491 +/- ##
==========================================
- Coverage 74.39% 74.39% -0.01%
==========================================
Files 768 768
Lines 107784 107784
==========================================
- Hits 80183 80181 -2
- Misses 27601 27603 +2
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 |
I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Random datagen is quite useful in not only connector, but also tests. For example, we can use this data generator to generate random data for micro bench mark. So we move it from connector crate to common crate.
Checklist
./risedev check
(or alias,./risedev c
)Refer to a related PR or issue link (optional)