m1a2st commented on code in PR #18420:
URL: https://github.com/apache/kafka/pull/18420#discussion_r1908705799
##########
clients/src/main/java/org/apache/kafka/common/utils/Sanitizer.java:
##########
@@ -24,14 +24,12 @@
import javax.management.ObjectName;
/**
- * Utility class for sanitizing/desanitizing/quoting values used in JMX metric
names
- * or as ZooKeeper node name.
+ * Utility class for sanitizing/desanitizing/quoting values used in JMX metric
names.
* <p>
- * User principals and client-ids are URL-encoded using ({@link
#sanitize(String)}
- * for use as ZooKeeper node names. User principals are URL-encoded in all
metric
- * names as well. All other metric tags including client-id are quoted if they
- * contain special characters using {@link #jmxSanitize(String)} when
- * registering in JMX.
+ * User principals and client-ids are URL-encoded using ({@link
#sanitize(String)}.
+ * User principals are URL-encoded in all metric names as well. All other
metric tags
Review Comment:
sure, thanks for your comments, addressed it.
--
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]
