frankvicky commented on code in PR #19104:
URL: https://github.com/apache/kafka/pull/19104#discussion_r1980828950
##########
clients/src/main/java/org/apache/kafka/common/security/auth/KafkaPrincipalBuilder.java:
##########
@@ -23,6 +23,10 @@
* Note that the {@link org.apache.kafka.common.Configurable} and {@link
java.io.Closeable}
* interfaces are respected if implemented. Additionally, implementations must
provide a
* default no-arg constructor.
+ *
+ * Note that when running in KRaft mode, custom implementations of {@code
KafkaPrincipalBuilder}
Review Comment:
nit: We could use `@link` here.
--
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]
