Closed ajm188 closed 1 year ago
@shlomi-noach For the "all" side on the CLI/rpc side at least, I was thinking we won't need separate commands (right now it's launch-all and launch <uuid>) instead similar to show where we take launch {<uuid>|'all'}. Thoughts?
launch-all
launch <uuid>
show
launch {<uuid>|'all'}
Agreed. We reuse the same command but the argument changes. This was also my re-implementation in vtctl: https://github.com/vitessio/vitess/pull/12963
vtctl
@shlomi-noach For the "all" side on the CLI/rpc side at least, I was thinking we won't need separate commands (right now it's
launch-all
andlaunch <uuid>
) instead similar toshow
where we takelaunch {<uuid>|'all'}
. Thoughts?