[ https://issues.apache.org/jira/browse/HBASE-28327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17810592#comment-17810592 ]
ASF GitHub Bot commented on HBASE-28327: ---------------------------------------- Apache-HBase commented on PR #5647: URL: https://github.com/apache/hbase/pull/5647#issuecomment-1909066183 :confetti_ball: **+1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 0m 36s | Docker mode activated. | | -0 :warning: | yetus | 0m 2s | Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck | ||| _ Prechecks _ | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 0m 12s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 3m 35s | master passed | | +1 :green_heart: | compile | 0m 23s | master passed | | +1 :green_heart: | shadedjars | 6m 27s | branch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 23s | master passed | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 12s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 3m 59s | the patch passed | | +1 :green_heart: | compile | 0m 26s | the patch passed | | +1 :green_heart: | javac | 0m 26s | the patch passed | | +1 :green_heart: | shadedjars | 6m 51s | patch has no errors when building our shaded downstream artifacts. | | +1 :green_heart: | javadoc | 0m 25s | the patch passed | ||| _ Other Tests _ | | +1 :green_heart: | unit | 0m 16s | hbase-metrics-api in the patch passed. | | +1 :green_heart: | unit | 0m 22s | hbase-metrics in the patch passed. | | | | 25m 16s | | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5647/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/hbase/pull/5647 | | Optional Tests | javac javadoc unit shadedjars compile | | uname | Linux 8d47f0204bba 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | dev-support/hbase-personality.sh | | git revision | master / 354a3a2774 | | Default Java | Eclipse Adoptium-11.0.17+8 | | Test Results | https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5647/1/testReport/ | | Max. process+thread count | 197 (vs. ulimit of 30000) | | modules | C: hbase-metrics-api hbase-metrics U: . | | Console output | https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5647/1/console | | versions | git=2.34.1 maven=3.8.6 | | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org | This message was automatically generated. > 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 > Reporter: Evelyn Boland > Priority: Major > > 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)