Skip to content
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 'SET CONFIG' syntax to change configs of TiKV/PD instances #16479

Closed
qw4990 opened this issue Apr 16, 2020 · 0 comments · Fixed by #16480
Closed

support 'SET CONFIG' syntax to change configs of TiKV/PD instances #16479

qw4990 opened this issue Apr 16, 2020 · 0 comments · Fixed by #16480
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@qw4990
Copy link
Contributor

qw4990 commented Apr 16, 2020

Development Task

After implementing this proposal(#13660), TiKV and PD support to dynamic configs.
Now we can get PD and TiKV instances' configs from PD according to pd-ctl.
Using pd-ctl is not convenient for our user, so we need to support to show configs by SQL.

After discussing, we decide to support these syntaxes:

  1. set config instance/type key=value

For example:

  1. set config tikv log.level='info'
  2. set config address log.level='info'

Our parser has already supported this syntax now.(pingcap/parser#768)

Teachability, Documentation, Adoption, Migration Strategy:

#13660

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant