cmccabe commented on code in PR #16669:
URL: https://github.com/apache/kafka/pull/16669#discussion_r1700513481
##########
core/src/main/scala/kafka/tools/StorageTool.scala:
##########
@@ -223,12 +172,20 @@ object StorageTool extends Logging {
action(storeTrue())
formatParser.addArgument("--release-version", "-r").
action(store()).
- help(s"A KRaft release version to use for the initial metadata.version.
The minimum is ${MetadataVersion.IBP_3_0_IV0}, the default is
${MetadataVersion.LATEST_PRODUCTION}")
+ help(s"The release version to use for the initial feature settings. The
minimum is " +
+ s"${MetadataVersion.IBP_3_0_IV0}; the default is
${MetadataVersion.LATEST_PRODUCTION}")
formatParser.addArgument("--feature", "-f").
- help("A feature upgrade we should perform, in feature=level format. For
example: `metadata.version=5`.").
- action(append());
-
- parser.parseArgsOrFail(args)
+ help("The setting to use for a specific feature, in feature=level
format. For example: `kraft.version=1`.").
+ action(append())
+ val reconfigurableQuorumOptions = formatParser.addMutuallyExclusiveGroup()
+ reconfigurableQuorumOptions.addArgument("--standalone", "-s").
+ help("Used to initialize a single-node quorum controller quorum.").
+ action(storeTrue())
+ reconfigurableQuorumOptions.addArgument("--initial-voters", "-I").
+ help("A list of controller quorum voter ids, directories, and
hostname:port pairs. For example:\n" +
+
"0-JEXY6aqzQY-32P5TStzaFg@localhost:8082,1-MvDxzVmcRsaTz33bUuRU6A@localhost:8083,2-07R5amHmR32VDA6jHkGbTA@localhost:8084\n").
Review Comment:
Good point. I added this text:
> The same values must be used to format all nodes.
--
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]
