lianetm commented on code in PR #18031:
URL: https://github.com/apache/kafka/pull/18031#discussion_r1869638746
##########
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:
it currently is "REQUIRED: The server(s) to connect to.". Maybe flipping it
to "The server(s) to connect to. REQUIRED for all options 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]
