-
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
plugin: support flush tidb plugin {name}
in tidb-server
#9320
Conversation
/run-all-tests |
1 similar comment
/run-all-tests |
LGTM |
/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
Codecov Report
@@ Coverage Diff @@
## master #9320 +/- ##
=========================================
- Coverage 67.2% 67.2% -0.01%
=========================================
Files 371 371
Lines 77572 77574 +2
=========================================
+ Hits 52132 52133 +1
Misses 20783 20783
- Partials 4657 4658 +1
Continue to review full report at Codecov.
|
What problem does this PR solve?
refer pingcap/parser#209 and prepose to #9136
What is changed and how it works?
impl was insprited by #9271
flush tidb plugin {name}
commanduse 2 phase init, plugin can Init before domain init and start watch loops after domain be ready.
Check List
Tests
script: (WIP)
Code changes
Side effects
Related changes
This change is