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

Incubating Program: Dynamic Configuration Change #89

Open
rleungx opened this issue Nov 21, 2019 · 3 comments
Open

Incubating Program: Dynamic Configuration Change #89

rleungx opened this issue Nov 21, 2019 · 3 comments
Labels
incubating-request/feature This is an Incubating Program request for a feature status/LGT1 Indicates that a PR has LGTM 1.

Comments

@rleungx
Copy link
Member

rleungx commented Nov 21, 2019

Incubating Program

Dynamic Configuration Change

Describe the feature or project you want to incubate:

Summary

This proposal proposes a unified way to manage the configuration options of TiDB, TiKV, and PD by storing them in PD and support dynamically change the configuration options by using the same way among the different components which can greatly improve usability.

Motivation

Here are some reasons why we need to do it:

  • For now, each component in TiDB cluster has its own configuration file, which is hard for management. we need a unified way to manage the configuration options of all components
  • Although some configuration options support dynamic modification, the operations need to learn a lot to use them properly since we have multiple entries, e.g., pd-ctl, tikv-ctl, and SQL, resulting in poor usability. For better usability, provide a unified way to modify them dynamically.

Your RFC/Proposal?

pingcap/tidb#13660

@rleungx rleungx added the incubating-request/feature This is an Incubating Program request for a feature label Nov 21, 2019
@winkyao
Copy link
Contributor

winkyao commented Nov 22, 2019

LGTM

@winkyao winkyao added the status/LGT1 Indicates that a PR has LGTM 1. label Nov 23, 2019
@qiuyesuifeng
Copy link
Member

LGTM

@zz-jason
Copy link
Member

the proposal still have comments to address, PTAL @rleungx

AndreMouche pushed a commit to AndreMouche/community that referenced this issue Nov 6, 2020
* Migrate to json for sig-coprocessor membership

Signed-off-by: Rustin-Liu <[email protected]>

* add co leaders

Signed-off-by: Rustin-Liu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incubating-request/feature This is an Incubating Program request for a feature status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Development

No branches or pull requests

4 participants