Skip to content

Commit 91bdc91

Browse files
visualYJDyuhaijun999
authored andcommitted
[fix][client] Add ChangeLogLevel subcommand
1 parent 72f6c7e commit 91bdc91

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

src/client_v2/coordinator.cc

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ void SetUpCoordinatorSubCommands(CLI::App &app) {
3535
SetUpGetRegionMap(app);
3636
SetUpGetJobList(app);
3737
SetUpCleanJobList(app);
38+
SetUpChangeLogLevel(app);
3839
// region commands
3940
SetUpAddPeerRegion(app);
4041
SetUpRemovePeerRegion(app);
@@ -60,7 +61,6 @@ void SetUpCoordinatorSubCommands(CLI::App &app) {
6061

6162
// balance
6263
SetUpEnableOrDisableBalance(app);
63-
6464
}
6565

6666
bool GetBrpcChannel(const std::string &location, brpc::Channel &channel) {
@@ -612,9 +612,9 @@ void RunChangeLogLevel(GetChangeLogLevelOption const &opt) {
612612
DINGO_LOG(WARNING) << "Fail to send request to : " << cntl.ErrorText();
613613
}
614614

615-
DINGO_LOG(INFO) << request.DebugString();
616-
DINGO_LOG(INFO) << ::dingodb::pb::node::LogLevel_descriptor()->FindValueByNumber(request.log_level())->name();
617-
DINGO_LOG(INFO) << response.DebugString();
615+
std::cout << request.DebugString() << std::endl;
616+
std::cout << ::dingodb::pb::node::LogLevel_descriptor()->FindValueByNumber(request.log_level())->name() << std::endl;
617+
std::cout << response.DebugString() << std::endl;
618618
}
619619

620620
void SetUpHello(CLI::App &app) {
@@ -2791,9 +2791,11 @@ void SetUpEnableOrDisableBalance(CLI::App &app) {
27912791
auto opt = std::make_shared<EnableOrDisableBalanceOptions>();
27922792
auto *cmd = app.add_subcommand("DisableBalance", "DisableBalance ")->group("Coordinator Command");
27932793
cmd->add_option("--coor_url", opt->coor_url, "Coordinator url, default:file://./coor_list");
2794-
cmd->add_option("--balance_leader", opt->enable_balance_leader, "Request parameter enable_balance_leader true or false")
2794+
cmd->add_option("--balance_leader", opt->enable_balance_leader,
2795+
"Request parameter enable_balance_leader true or false")
27952796
->required();
2796-
cmd->add_option("--balance_region", opt->enable_balance_region, "Request parameter enable_balance_region true or false")
2797+
cmd->add_option("--balance_region", opt->enable_balance_region,
2798+
"Request parameter enable_balance_region true or false")
27972799
->required();
27982800
cmd->callback([opt]() { RunEnableOrDisableBalance(*opt); });
27992801
}

0 commit comments

Comments
 (0)