[ https://issues.apache.org/jira/browse/HBASE-28327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17811063#comment-17811063 ]
Hudson commented on HBASE-28327: -------------------------------- Results for branch branch-2.5 [build #470 on builds.a.o|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/470/]: (x) *{color:red}-1 overall{color}* ---- details (if available): (/) {color:green}+1 general checks{color} -- For more information [see general report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/470/General_20Nightly_20Build_20Report/] (x) {color:red}-1 jdk8 hadoop2 checks{color} -- For more information [see jdk8 (hadoop2) report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/470/JDK8_20Nightly_20Build_20Report_20_28Hadoop2_29/] (x) {color:red}-1 jdk8 hadoop3 checks{color} -- For more information [see jdk8 (hadoop3) report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/470/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 jdk11 hadoop3 checks{color} -- For more information [see jdk11 report|https://ci-hbase.apache.org/job/HBase%20Nightly/job/branch-2.5/470/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/] (/) {color:green}+1 source release artifact{color} -- See build output for details. (/) {color:green}+1 client integration test{color} > Add remove(String key, Metric metric) method to MetricRegistry interface > ------------------------------------------------------------------------ > > Key: HBASE-28327 > URL: https://issues.apache.org/jira/browse/HBASE-28327 > Project: HBase > Issue Type: Improvement > Components: metrics > Reporter: Evelyn Boland > Assignee: Evelyn Boland > Priority: Major > Fix For: 2.6.0, 2.5.8, 3.0.0-beta-2 > > > Add a `remove(String name, Metric metric)` method to the `MetricRegistry` > interface. Right now the interface only contains a `remove(String name)` > method. > This additional remove method will give users the power to remove a `Metric` > with the specified `name` from the metric registry if and only if the > provided `metric` matches the object in the registry. > Implementing the new `remove(String name, Metric metric)` should be straight > forward because the `MetricRegistryImpl` class stores metrics in a > `ConcurrentMap`, which already contains a `remove(Object key, Object value)` > method > This change will not be a breaking one because the interface is marked with > `@IntefaceStability.Evolving` > [~rmdmattingly] -- This message was sent by Atlassian Jira (v8.20.10#820010)