[
https://issues.apache.org/jira/browse/FLINK-5261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15722282#comment-15722282
]
ASF GitHub Bot commented on FLINK-5261:
---------------------------------------
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/2944#discussion_r90864749
--- Diff:
flink-metrics/flink-metrics-dropwizard/src/test/java/org/apache/flink/dropwizard/ScheduledDropwizardReporterTest.java
---
@@ -124,6 +129,89 @@ public void testAddingMetrics() throws
NoSuchFieldException, IllegalAccessExcept
metricRegistry.shutdown();
}
+ @Test
--- End diff --
Can you add a high level description about what this test does please?
Otherwise this looks good.
> ScheduledDropwizardReporter does not properly clean up metrics
> --------------------------------------------------------------
>
> Key: FLINK-5261
> URL: https://issues.apache.org/jira/browse/FLINK-5261
> Project: Flink
> Issue Type: Bug
> Components: Metrics
> Affects Versions: 1.2.0
> Reporter: Chesnay Schepler
> Assignee: Chesnay Schepler
> Priority: Blocker
> Fix For: 1.2.0
>
>
> The ScheduleDropWizardReporter does not have a separate branch for meters in
> notifyOfRemovedMetric.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
