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
store/helper, infoschema: add mem table for tikv store status #10248
Conversation
{"REGION_SIZE", mysql.TypeLonglong, 21, 0, nil, nil}, | ||
{"START_TS", mysql.TypeDatetime, 0, 0, nil, nil}, | ||
{"LAST_HEARTBEAT_TS", mysql.TypeDatetime, 0, 0, nil, nil}, | ||
{"UPTIME", mysql.TypeVarchar, 64, 0, nil, nil}, |
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.
uptime
should be duration actually. But since mysql's Time
type's data range is just [-838 hour, 838 hour]. I think it's not very suitable to use it here.
/rebuild |
/run-all-tests |
/run-all-tests |
@winoros Please fix the UT. |
/run-all-tests |
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.
LGTM
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.
LGTM
… into tikv-status-mem-table
Codecov Report
@@ Coverage Diff @@
## master #10248 +/- ##
================================================
- Coverage 77.8232% 77.7727% -0.0505%
================================================
Files 410 410
Lines 84372 84446 +74
================================================
+ Hits 65661 65676 +15
- Misses 13811 13865 +54
- Partials 4900 4905 +5 |
Codecov Report
@@ Coverage Diff @@
## master #10248 +/- ##
===============================================
- Coverage 77.941% 77.8856% -0.0554%
===============================================
Files 409 409
Lines 83866 83950 +84
===============================================
+ Hits 65366 65385 +19
- Misses 13636 13697 +61
- Partials 4864 4868 +4 |
@lamxTyler fixed |
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.
LGTM
What problem does this PR solve?
Add a mem table to show the tikv status.
What is changed and how it works?
use pd api to get the information.
The table is shown like below.
Check List
Tests
Code changes
Related changes