search

tikv / raft-engine

A persistent storage engine for Multi-Raft log
Apache License 2.0
565 stars 88 forks source link

Replace app builder with derive macros in stress tool #155

Closed tabokie closed 2 years ago

tabokie commented 2 years ago

Similar to ctl tool, we can replace the use of clap with structopt in stress. It can greatly simplify the code.

skyzh commented 2 years ago

You can use clap-v3.0 beta, which shares nearly the same usage as structopt. We have refactored from structopt to clap-v3.0 in TiKV before

ztelur commented 2 years ago

Hi, I want to do this work, plz assign it to me

tabokie commented 2 years ago

Thanks! /assign @ztelur

ztelur commented 2 years ago

should we use clap-v3.0 or structopt?

tabokie commented 2 years ago

clap is fine, you can update ctl to use clap too.