dajac commented on a change in pull request #11502:
URL: https://github.com/apache/kafka/pull/11502#discussion_r750109834
##########
File path: core/src/main/scala/kafka/admin/AclCommand.scala
##########
@@ -499,13 +502,17 @@ object AclCommand extends Logging {
.describedAs("command-config")
.ofType(classOf[String])
- val authorizerOpt = parser.accepts("authorizer", "Fully qualified class
name of the authorizer, defaults to kafka.security.authorizer.AclAuthorizer.")
+ val authorizerOpt = parser.accepts("authorizer", "DEPRECATED: Fully
qualified class name of " +
+ "the authorizer, which defaults to
kafka.security.authorizer.AclAuthorizer if --bootstrap-server is not provided.
" +
+ AclCommand.AuthorizerDeprecationMessage)
.withRequiredArg
.describedAs("authorizer")
.ofType(classOf[String])
- val authorizerPropertiesOpt = parser.accepts("authorizer-properties",
"REQUIRED: properties required to configure an instance of Authorizer. " +
- "These are key=val pairs. For the default authorizer the example values
are: zookeeper.connect=localhost:2181")
+ val authorizerPropertiesOpt = parser.accepts("authorizer-properties",
"DEPRECATED: The " +
+ "properties required to configure an instance of the Authorizer
specified by --authorizer. " +
+ "These are key=val pairs. For the default authorizer, example values
are: zookeeper.connect=localhost:2181." +
Review comment:
nit: Need to add a space after the `.`.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
