-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
infochema, server: export region_id in hot_region table #10982
Conversation
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #10982 +/- ##
===========================================
Coverage 81.2626% 81.2626%
===========================================
Files 423 423
Lines 90781 90781
===========================================
Hits 73771 73771
Misses 11705 11705
Partials 5305 5305 |
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
/run-all-tests |
/run-all-tests |
/run-unit-test |
@lysu please fix test |
/run-all-tests |
/run-unit-test |
1 similar comment
/run-unit-test |
/run-all-tests |
@imtbkcat PTAL again thx~ |
/run-all-tests |
/run-unit-test |
/run-all-tests |
/rebuild |
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?
REGION_ID
toinformation_schema.TIDB_HOT_REGIONS
so we can join other table to find sth like "hot store name", "hot key ranges".TIKV_REGION_PEERS
What is changed and how it works?
just as above described
Check List
Tests
run sysbench prepare will saw this:
Code changes
Side effects
Related changes
This change is