-
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
*: Support hot-reloading option enable-arrow #12621
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12621 +/- ##
===========================================
Coverage 79.8577% 79.8577%
===========================================
Files 462 462
Lines 105723 105723
===========================================
Hits 84428 84428
Misses 15013 15013
Partials 6282 6282 |
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
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
Your auto merge job has been accepted, waiting for 12769 |
/run-all-tests |
What problem does this PR solve?
Improve the usability for the
enable-arrow
What is changed and how it works?
Support hot-reloading option
enable-arrow
.Check List
Tests
step 1: select the current value:
select @@tidb_enable_arrow;
step 2: change the value of
tidb_enable_arrow
in the config filestep 3: reload the config
curl -X GET http://127.0.0.1:10080/reload-config
step 4: open a new mysql-client, re-select the value:
select @@tidb_enable_arrow;
Code changes
Side effects
Related changes
Release note