-
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
x-server: remove the X Server #9038
Conversation
/run-all-tests |
LGTM |
/run-integraion-common-test tidb-test=pr/718 |
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-integration-common-test tidb-test=pr/718 |
/run-all-tests |
/run-unit-test |
Codecov Report
@@ Coverage Diff @@
## master #9038 +/- ##
==========================================
+ Coverage 67.25% 67.26% +<.01%
==========================================
Files 371 371
Lines 76195 76195
==========================================
+ Hits 51245 51250 +5
+ Misses 20415 20412 -3
+ Partials 4535 4533 -2
Continue to review full report at Codecov.
|
What problem does this PR solve?
X Server is never used in TiDB. It is better to implement it as a plugin as MySQL, due to the plugin framework is supported in TiDB.
What is changed and how it works?
Remove the X Server
Check List
Tests