jsancio commented on code in PR #12921: URL: https://github.com/apache/kafka/pull/12921#discussion_r1035377520
########## clients/src/main/java/org/apache/kafka/common/config/internals/BrokerSecurityConfigs.java: ########## @@ -61,10 +61,10 @@ public class BrokerSecurityConfigs { public static final String SASL_KERBEROS_PRINCIPAL_TO_LOCAL_RULES_DOC = "A list of rules for mapping from principal " + "names to short names (typically operating system usernames). The rules are evaluated in order and the " + "first rule that matches a principal name is used to map it to a short name. Any later rules in the list are " + - "ignored. By default, principal names of the form {username}/{hostname}@{REALM} are mapped to {username}. " + - "For more details on the format please see <a href=\"#security_authz\"> security authorization and acls</a>. " + - "Note that this configuration is ignored if an extension of KafkaPrincipalBuilder is provided by the " + - "<code>" + PRINCIPAL_BUILDER_CLASS_CONFIG + "</code> configuration."; + "ignored. By default, principal names of the form <code>{username}/{hostname}@{REALM}</code> are mapped + + "to <code>{username}</code>. For more details on the format please see <a href=\"#security_authz\"> + + "security authorization and acls</a>. Note that this configuration is ignored if an extension of + Review Comment: These 3 lines are missing and end quote. E.g. ```java "ignored. By default, principal names of the form <code>{username}/{hostname}@{REALM}</code> are mapped " + "to <code>{username}</code>. For more details on the format please see <a href=\"#security_authz\"> " + "security authorization and acls</a>. Note that this configuration is ignored if an extension of " + ``` -- 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