Skip to content

Commit

Permalink
[#8744] docdb: Fix yb-admin arg count for disable_tablet_splitting
Browse files Browse the repository at this point in the history
Summary: disable_tablet_splitting requires 2 arguments, but we currently only validate that at least one is passed.

Test Plan: Run disable_tablet_splitting with yb-admin with 1 argument. The command should fail.

Reviewers: jhe

Reviewed By: jhe

Subscribers: ybase

Differential Revision: https://phorge.dev.yugabyte.com/D32728
  • Loading branch information
SrivastavaAnubhav committed Feb 29, 2024
1 parent eaf4896 commit e1510a1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/tools/yb-admin_cli.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1107,7 +1107,7 @@ Status split_tablet_action(const ClusterAdminCli::CLIArguments& args, ClusterAdm
const auto disable_tablet_splitting_args = "<disable_duration_ms> <feature_name>";
Status disable_tablet_splitting_action(
const ClusterAdminCli::CLIArguments& args, ClusterAdminClient* client) {
if (args.size() < 1) {
if (args.size() < 2) {
return ClusterAdminCli::kInvalidArguments;
}
const int64_t disable_duration_ms = VERIFY_RESULT(CheckedStoll(args[0]));
Expand Down

0 comments on commit e1510a1

Please sign in to comment.