AndrewJSchofield commented on code in PR #17243:
URL: https://github.com/apache/kafka/pull/17243#discussion_r1812962292
##########
core/src/main/scala/kafka/admin/ConfigCommand.scala:
##########
@@ -299,7 +299,7 @@ object ConfigCommand extends Logging {
val configsToBeAdded = opts.options.valueOf(opts.addConfig)
.split("," + pattern)
.map(_.split("""\s*=\s*""" + pattern, -1))
- require(configsToBeAdded.forall(config => config.length == 2), "Invalid
entity config: all configs to be added must be in the format \"key=val\".")
+ require(configsToBeAdded.forall(config => config.length == 2), "Invalid
entity config: all configs to be added must be in the format of \"key=val\". If
val is in the format symbol 'val1,val2', you can use \"key=[val1,val2]\".")
Review Comment:
I suggest the following: `"Invalid entity config: all configs to be added
must be in the format \"key=val\" or \"key=[val1,val2]\" to group values which
contain commas."`
--
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]
