Skip to content

Conversation

@acelyc111
Copy link
Member

@acelyc111 acelyc111 commented Apr 4, 2020

Ref #3268
In the past, when we want to modify some BE configs, we have to modify be.conf and then restart BE.
This patch provides a way to modify configs in the type of 'threshold', 'interval', 'enable flag'
when BE is running without restarting it.
You can update a single config once by BE's http API: be_host:be_http_port/api/update_config?config_name=new_value

…ing it

In the past, when we want to modify some BE configs, we have to modify be.conf and then restart BE.
This patch provides a way to modify configs in the type of 'threshold', 'interval', 'enable flag'
when BE is running without restarting it.
You can update a single config once by BE's http API: `be_host:be_http_port/api/update_config?config_name=new_value`
@imay
Copy link
Contributor

imay commented Apr 6, 2020

@acelyc111 This is a wonderful job, could you please to create an issue to track this feature and relate this PR to it?

@acelyc111
Copy link
Member Author

@acelyc111 This is a wonderful job, could you please to create an issue to track this feature and relate this PR to it?

OK, #3268

Copy link
Contributor

@imay imay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@imay imay merged commit 8fc284d into apache:master Apr 8, 2020
@EmmyMiao87 EmmyMiao87 mentioned this pull request Aug 17, 2020
acelyc111 added a commit to acelyc111/incubator-doris that referenced this pull request Jan 20, 2021
…ing (apache#3264)

In the past, when we want to modify some BE configs, we have to modify be.conf and then restart BE.
This patch provides a way to modify configs in the type of 'threshold', 'interval', 'enable flag'
when BE is running without restarting it.
You can update a single config once by BE's http API: `be_host:be_http_port/api/update_config?config_name=new_value`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants