-
Notifications
You must be signed in to change notification settings - Fork 780
Add tokio runtime memory tracker #2356
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
Add tokio runtime memory tracker #2356
Conversation
Thanks for the contribution! Please review the labels and make any necessary changes. |
Love it XDDD |
# Conflicts: # common/mem/mem-allocator/Cargo.toml
|
Codecov Report
@@ Coverage Diff @@
## main #2356 +/- ##
=====================================
- Coverage 71% 71% -1%
=====================================
Files 636 638 +2
Lines 35853 35941 +88
=====================================
+ Hits 25515 25576 +61
- Misses 10338 10365 +27
Continue to review full report at Codecov.
|
# Conflicts: # query/src/bin/databend-query.rs
Love it \XDDD |
/lgtm refreshing! |
Approved! Thank you for the PR @zhang2014 |
Curious about how to accumulate the session memory cost 🤔 : As it is a tree of threads. |
CI Passed |
There is always a one-to-one mapping between tokio runtime and session in the databend query |
I hereby agree to the terms of the CLA available at: https://databend.rs/policies/cla/
Summary
Add tokio runtime memory tracker
Changelog
Related Issues
#2299 (comment)
Fixes #1148
Test Plan
Unit Tests
Stateless Tests