-
Notifications
You must be signed in to change notification settings - Fork 772
ci: Try to reduce unittest memory usage #3929
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
Conversation
Signed-off-by: Xuanwo <[email protected]>
Thanks for the contribution! Please review the labels and make any necessary changes. |
1 similar comment
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/G62pWacsEW8atTAoGPdjS1FM7nVa [Deployment for 0d62561 canceled] |
I plan to rerun this PR's test_unit three time to make sure it really works. |
Co-authored-by: Chojan Shang <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #3929 +/- ##
=====================================
Coverage 56% 56%
=====================================
Files 800 801 +1
Lines 42754 42714 -40
=====================================
- Hits 24350 24331 -19
+ Misses 18404 18383 -21
Continue to review full report at Codecov.
|
Signed-off-by: Xuanwo [email protected]
I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
This PR intends to reduce memory usage during compiling.
It's a workaround inspired by rust-lang/rust#82406 (comment).
Hoping this can resolve the failed tests for databend.
Changelog
Test Plan
Unit Tests
Stateless Tests