You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
According to the prompt of the command line, I tried to start the service with the -s parameter:
./target/debug/ballista-scheduler --help
Usage: ./target/debug/ballista-scheduler [ARGUMENTS...]
Arguments:
-s, --scheduler-policy The scheduing policy for the scheduler, see
TaskSchedulingPolicy::variants() for options.
Default: PushStaged
the result was an error:
./target/debug/ballista-scheduler -s PushStaged
Error: Failed to parse argument '--scheduler-policy': Invalid variant: PushStaged.
Hint: the value must be The scheduler policy for the scheduler.
Describe the solution you'd like
In this code:
// an enum used to configure the scheduler policy// needs to be visible to code generated by configure_me#[derive(Clone,ArgEnum,Copy,Debug, serde::Deserialize)]pubenumTaskSchedulingPolicy{PullStaged,PushStaged,}
clap ArgEnum derive macro generate possible values pull-staged, push-staged, not PullStaged, PushStaged
There are two solutions:
Modify the help doc to make it clear that the -s parameter values are pull-staged, push-staged.
Modify the TaskSchedulingPolicy enum and add #[clap(rename_all = "camel")], it generate possible values pullStaged, pushStaged
I can submit a PR for this, but need some suggestions.
The text was updated successfully, but these errors were encountered:
Describe the bug
According to the prompt of the command line, I tried to start the service with the -s parameter:
the result was an error:
Describe the solution you'd like
In this code:
clap
ArgEnum
derive macro generate possible valuespull-staged, push-staged
, notPullStaged, PushStaged
There are two solutions:
-s
parameter values arepull-staged, push-staged
.TaskSchedulingPolicy
enum and add#[clap(rename_all = "camel")]
, it generate possible valuespullStaged, pushStaged
I can submit a PR for this, but need some suggestions.
The text was updated successfully, but these errors were encountered: