[ https://issues.apache.org/jira/browse/KAFKA-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14267397#comment-14267397 ]
Vladimir Tretyakov commented on KAFKA-1723: ------------------------------------------- Hi, thx for patch, one question from me, in patch I see something like that: {code} + Map<String, String> tags = new LinkedHashMap<String, String>(); + tags.put("clientId", clientId); + tags.put("topic", topic); + - topicRecordCount.add("topic." + topic + ".record-send-rate", new Rate()); + topicRecordCount.add("topic." + topic + ".record-send-rate", "ProducerTopicMetrics", tags, new Rate()); {code} Note that "("topic." + topic + ".record-send-rate"" part and tags both contain "topic". Can we avoid duplication and use in all places something like: topicRecordCount.add("record-send-rate", "ProducerTopicMetrics", tags, new Rate()); ? Thx. > make the metrics name in new producer more standard > --------------------------------------------------- > > Key: KAFKA-1723 > URL: https://issues.apache.org/jira/browse/KAFKA-1723 > Project: Kafka > Issue Type: Sub-task > Components: clients > Affects Versions: 0.8.2 > Reporter: Jun Rao > Assignee: Manikumar Reddy > Priority: Blocker > Fix For: 0.8.2 > > Attachments: KAFKA-1723.patch > > > The jmx name in the new producer looks like the following: > kafka.producer.myclientid:type=mytopic > However, this can be ambiguous since we allow "." in client id and topic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)