walterddr opened a new issue #7710: URL: https://github.com/apache/pinot/issues/7710
#7673 outlines the major step to retire args4j and replace it with picocli. However there are a couple of follow-ups that we need to do to ensure better error messaging and raising earlier argument parsing exception 1. Add mutually exclusive option check for now. (e.g. -configFile cannot be used together with explicit config settings, but we dont error out for now) 2. Add enum checker (e.g. -fileFormat accepts only ARVO/JSON/CSV... but not arbitrary string) 3. Usage of `org.apache.pinot.tools.Command` and `org.apache.pinot.tools.AbstractBaseCommand` can be simplify because picocli natively supports --help and --version redirect. 4. better printout for help messages and example. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
