-
Notifications
You must be signed in to change notification settings - Fork 6.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
Added reloading storage configuration from configuration file #8594
Added reloading storage configuration from configuration file #8594
Conversation
This is exciting. Can you share more about the intention here and what use cases you want to support? Ie. will it be possible to just remove a disk and ClickHouse will run part checks/repair for replicated tables and missing parts? |
@excitoon Any updates?
If parts are not available or not readable, ClickHouse will download them from replicas. Removing a disk and removing it from the configuration will work. |
boost::shared_ptr - looks strange. |
(rebased but boost::shared_ptr remains) |
@alexey-milovidov I did not use MultiVersion because |
@excitoon |
@BayoNet The configuration will be updated automatically when user has changed the config file. |
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Changelog category (leave one):
Changelog entry (up to few sentences, required except for Non-significant/Documentation categories):
Added reloading storage configuration from configuration file.