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
[Feature] add sys.fe_memory_usage table #40464
Conversation
Signed-off-by: Astralidea <astralidea@163.com>
fe/fe-core/src/main/java/com/starrocks/catalog/system/sys/SysFeMemoryUsage.java
Show resolved
Hide resolved
Signed-off-by: Xueyan Li <astralidea@163.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.
Add some sql-test to cover this new sys table
fe/fe-core/src/main/java/com/starrocks/catalog/system/sys/SysFeMemoryUsage.java
Show resolved
Hide resolved
Quality Gate passedThe SonarCloud Quality Gate passed, but some issues were introduced. 5 New issues |
[FE Incremental Coverage Report]✅ pass : 44 / 47 (93.62%) file detail
|
[BE Incremental Coverage Report]✅ pass : 34 / 34 (100.00%) file detail
|
https://github.com/Mergifyio backport branch-3.2 |
✅ Backports have been created
|
Signed-off-by: Astralidea <astralidea@163.com> Signed-off-by: Xueyan Li <astralidea@163.com> (cherry picked from commit 49d4380) # Conflicts: # be/src/exec/schema_scanner.cpp # fe/fe-core/src/main/java/com/starrocks/catalog/system/SystemId.java # gensrc/thrift/Descriptors.thrift
Why I'm doing:
Although we have log monitoring, for some environments, users may not have a way to directly obtain log information and view it, so we expose a metadata table to check the current memory statistics.
What I'm doing:
Fixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: