Skip to content

Fix cli argument in clickhouse-server.init#24449

Merged
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
vdimir:CLICKHOUSE-5062
May 25, 2021
Merged

Fix cli argument in clickhouse-server.init#24449
alexey-milovidov merged 3 commits intoClickHouse:masterfrom
vdimir:CLICKHOUSE-5062

Conversation

@vdimir
Copy link
Copy Markdown
Member

@vdimir vdimir commented May 24, 2021

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 24, 2021
@vdimir
Copy link
Copy Markdown
Member Author

vdimir commented May 24, 2021

Or maybe better to use po::bool_switch here ?

("force", po::value<bool>()->default_value(false), "Stop with KILL signal instead of TERM")

@alexey-milovidov alexey-milovidov self-assigned this May 24, 2021
@alexey-milovidov alexey-milovidov merged commit ea97eee into ClickHouse:master May 25, 2021
@vdimir vdimir deleted the CLICKHOUSE-5062 branch May 25, 2021 08:00
@vdimir vdimir added pr-backport Changes, backported to release branch. Do not use manually - automated use only! pr-must-backport Pull request should be backported intentionally. Use this label with great care! and removed pr-backport Changes, backported to release branch. Do not use manually - automated use only! labels May 25, 2021
vdimir pushed a commit that referenced this pull request May 25, 2021
Fix cli argument in clickhouse-server.init

(cherry picked from commit ea97eee)
kitaisreal added a commit that referenced this pull request May 25, 2021
@vdimir vdimir mentioned this pull request May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants