-
Notifications
You must be signed in to change notification settings - Fork 761
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
chore: routine maintenance #4064
Conversation
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
Thanks for the contribution! Please review the labels and make any necessary changes. |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/databend/databend/G7rgbD9nLb6Yyp2RsyABEQiaw546 [Deployment for 6136232 canceled] |
Codecov Report
@@ Coverage Diff @@
## main #4064 +/- ##
======================================
Coverage 57% 57%
======================================
Files 830 829 -1
Lines 44352 44231 -121
======================================
+ Hits 25486 25533 +47
+ Misses 18866 18698 -168
Continue to review full report at Codecov.
|
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
Signed-off-by: Chojan Shang <[email protected]>
todo:
I will be opening a new PR to handle it, probably to be finished tomorrow. |
LGTM, nice and slick. |
Wait for another reviewer approval |
/lgtm |
CI Passed |
I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
Changelog
Related Issues
Fixes #4062
Test Plan
Unit Tests
Stateless Tests