[
https://issues.apache.org/jira/browse/FLINK-4192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15387666#comment-15387666
]
ASF GitHub Bot commented on FLINK-4192:
---------------------------------------
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/2226
Having `MetricsConfig` extend `Properties` mitigates the issue to some
extend.
I know I am very adamant on not having an extra config object. It's because
I heard it a bunch of times that the plethora of config types annoy developers
(in general, not Flink specific) when every tool/system/library introduces
their own config types. So it would be nice if Flink did not introduce more
than one custom config type.
> Move Metrics API to separate module
> -----------------------------------
>
> Key: FLINK-4192
> URL: https://issues.apache.org/jira/browse/FLINK-4192
> Project: Flink
> Issue Type: Improvement
> Components: Metrics
> Affects Versions: 1.1.0
> Reporter: Chesnay Schepler
> Assignee: Chesnay Schepler
> Fix For: 1.1.0
>
>
> All metrics code currently resides in flink-core. If a user implements a
> reporter and wants a fat jar it will now have to include the entire
> flink-core module.
> Instead, we could move several interfaces into a separate module.
> These interfaces to move include:
> * Counter, Gauge, Histogram(Statistics)
> * MetricGroup
> * MetricReporter, Scheduled, AbstractReporter
> In addition a new MetricRegistry interface will be required as well as a
> replacement for the Configuration.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
