[
https://issues.apache.org/jira/browse/KAFKA-1481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14113641#comment-14113641
]
Otis Gospodnetic commented on KAFKA-1481:
-----------------------------------------
bq. if you know the clientId, you can parse the rest of the string.
I think this makes parsers unnecessarily difficult to write and probably harder
to maintain. Parsers have to remember things this way.
bq. If this patch is going to cause existing users that are not having a
problem issues
I think it won't cause them issues. I *think* they would just need to change
which delimiter character they use to break MBean names into parts.
bq. If it is causing existing users issues by not patching it
I think it is causing issues to anyone who has any of the multiple characters
Kafka uses as delimiters in their hostnames or topic names. So I think these
people have issues when they first try parsing stuff from JMX, and then they
have to go modify their hostnames, topic names and such.
> Stop using dashes AND underscores as separators in MBean names
> --------------------------------------------------------------
>
> Key: KAFKA-1481
> URL: https://issues.apache.org/jira/browse/KAFKA-1481
> Project: Kafka
> Issue Type: Bug
> Components: core
> Affects Versions: 0.8.1.1
> Reporter: Otis Gospodnetic
> Labels: patch
> Fix For: 0.8.2
>
> Attachments: KAFKA-1481_2014-06-06_13-06-35.patch
>
>
> MBeans should not use dashes or underscores as separators because these
> characters are allowed in hostnames, topics, group and consumer IDs, etc.,
> and these are embedded in MBeans names making it impossible to parse out
> individual bits from MBeans.
> Perhaps a pipe character should be used to avoid the conflict.
> This looks like a major blocker because it means nobody can write Kafka 0.8.x
> monitoring tools unless they are doing it for themselves AND do not use
> dashes AND do not use underscores.
> See: http://search-hadoop.com/m/4TaT4lonIW
--
This message was sent by Atlassian JIRA
(v6.2#6252)
