dajac commented on code in PR #18031:
URL: https://github.com/apache/kafka/pull/18031#discussion_r1869622426
##########
tools/src/main/java/org/apache/kafka/tools/consumer/group/ConsumerGroupCommandOptions.java:
##########
@@ -210,7 +216,9 @@ private ConsumerGroupCommandOptions(String[] args) {
void checkArgs() {
CommandLineUtils.maybePrintHelpOrVersion(this, "This tool helps to
list all consumer groups, describe a consumer group, delete consumer group
info, or reset consumer group offsets.");
- CommandLineUtils.checkRequiredArgs(parser, options,
bootstrapServerOpt);
+ if (!options.has(validateRegexOpt)) {
+ CommandLineUtils.checkRequiredArgs(parser, options,
bootstrapServerOpt);
+ }
Review Comment:
yeah, i considered doing this but i did not like it. how would you state it?
"required except for --validate-regex"?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
