GitHub user zentol opened a pull request:
https://github.com/apache/flink/pull/2226
[FLINK-4192] - Move Metrics API to separate module
This PR moves several metrics-related classes into a new module.
The new module is located under `flink-metrics/flink-metrics-core`.
Most of the changes are related to
* renaming of `MetricRegistry` to `InternalMetricRegistry`
* a new `MetricRegistry` interface was added
* replacing usages of `Configuration` with a new `MetricConfig`
Changes unrelated to the above are as follows:
* obviously, creating a new module and adjusting dependencies
* some tests were refactored to properly close the registry
* `getScopeComponents/-String()` was moved from `AbstractMetricGroup` to
`MetricGroup`
* fixed a small typo in `Scheduled` interface
I have verified that it compiles and works for example.The tests in
flink-core pass as well, I'll leave the rest to travis.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zentol/flink metrics_4th_of_july
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/2226.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2226
----
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
