[ 
https://issues.apache.org/jira/browse/KAFKA-1481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171371#comment-14171371
 ] 

Vladimir Tretyakov edited comment on KAFKA-1481 at 10/14/14 7:09 PM:
---------------------------------------------------------------------

Hi, Jun, 

Thx that you found a time to look at patch, I've added another regarding your 
suggestion, now in JMX you will see (hope it is what you need):
{code}
"kafka.server":type="FetcherStats",name="RequestsPerSec",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",brokerHost="wawanawna",brokerPort="9092"
"kafka.server":type="FetcherLagMetrics",name="ConsumerLag",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",brokerHost="wawanawna",brokerPort="9092",topic="spm_topic",partitionId="0"
"kafka.consumer":type="ZookeeperConsumerConnector",name="OwnedPartitionsCount",clientId="af_servers",groupId="af_servers",topic="spm_topic"
"kafka.consumer":type="FetchRequestAndResponseMetrics",name="FetchResponseSize",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",allBrokers="true"
{code}

{quote}
Also, your patch doesn't seem to apply to latest trunk.
git apply ~/Downloads/KAFKA-1481_2014-10-13_18-23-35.patch 
error: core/src/main/scala/kafka/common/ClientIdTopic.scala: No such file or 
directory
{quote}
Interesting...
1. I've worked with 0.8.2 branch, not trunk
2. ClientIdTopic.scala - I've added this file (must be in patch). 

Added 2 equal patches (created them in different way):
1. 'git diff' (KAFKA-1481_2014-10-14_21-53-35.patch)
2. With help from IDEA IDE. (KAFKA-1481_IDEA_IDE_2014-10-14_21-53-35.patch)

Try please, maybe one of them will work

Thx again.



was (Author: vladimir.tretyakov):
Hi, Jun, 

Thx that you found a time to look at patch, I've added another regarding your 
suggestion, now in JMX you will see (hope it is what you need):
{code}
"kafka.server":type="FetcherStats",name="RequestsPerSec",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",brokerHost="wawanawna",brokerPort="9092"
"kafka.server":type="FetcherLagMetrics",name="ConsumerLag",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",brokerHost="wawanawna",brokerPort="9092",topic="spm_topic",partitionId="0"
"kafka.consumer":type="ZookeeperConsumerConnector",name="OwnedPartitionsCount",clientId="af_servers",groupId="af_servers",topic="spm_topic"
"kafka.consumer":type="FetchRequestAndResponseMetrics",name="FetchResponseSize",clientId="af_servers",threadName="ConsumerFetcherThread",groupId="af_servers",consumerHostName="wawanawna",timestamp="1413306414731",uuid="4624cb0f",fetcherId="0",sourceBrokerId="0",allBrokers="true"
{code}

{quote}
Also, your patch doesn't seem to apply to latest trunk.
git apply ~/Downloads/KAFKA-1481_2014-10-13_18-23-35.patch 
error: core/src/main/scala/kafka/common/ClientIdTopic.scala: No such file or 
directory
{quote}
Interesting...
1. I've worked with 0.8.2 branch, not trunk
2. ClientIdTopic.scala - I've added this file (must be in patch). 

Added 2 equal patches (created them in different way):
1. 'git diff' 
2. With help from IDEA IDE.

Try please, maybe one of them will work

Thx again.


> 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
>            Priority: Critical
>              Labels: patch
>             Fix For: 0.8.2
>
>         Attachments: KAFKA-1481_2014-06-06_13-06-35.patch, 
> KAFKA-1481_2014-10-13_18-23-35.patch, KAFKA-1481_2014-10-14_21-53-35.patch, 
> KAFKA-1481_IDEA_IDE_2014-10-14_21-53-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.3.4#6332)

Reply via email to