[
https://issues.apache.org/jira/browse/KAFKA-6307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16301542#comment-16301542
]
ASF GitHub Bot commented on KAFKA-6307:
---------------------------------------
tedyu commented on issue #4307: KAFKA-6307 mBeanName should be removed before
returning from JmxReporter#removeAttribute()
URL: https://github.com/apache/kafka/pull/4307#issuecomment-353617134
Previously ```int countOfMetric(KafkaMetric metric)``` was added to
JmxReporter but @ijuma doesn't want to expose such method.
@ijuma
Can you confirm @satishd 's suggestion ?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> mBeanName should be removed before returning from
> JmxReporter#removeAttribute()
> -------------------------------------------------------------------------------
>
> Key: KAFKA-6307
> URL: https://issues.apache.org/jira/browse/KAFKA-6307
> Project: Kafka
> Issue Type: Bug
> Reporter: Ted Yu
> Assignee: siva santhalingam
>
> JmxReporter$KafkaMbean showed up near the top in the first histo output from
> KAFKA-6199.
> In JmxReporter#removeAttribute() :
> {code}
> KafkaMbean mbean = this.mbeans.get(mBeanName);
> if (mbean != null)
> mbean.removeAttribute(metricName.name());
> return mbean;
> {code}
> mbeans.remove(mBeanName) should be called before returning.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
