GitHub user zentol opened a pull request:
https://github.com/apache/flink/pull/2979
[FLINK-5307] [metrics] Log reporter configuration
With this PR the configuration for every reporter will be logged. Twice.
First, we log the `Properties` object that contains the configuration for
each reporter _before_ passing it to the reporter. This allows users to detect
missing properties that are result of types in the reporter name or preceding
parts of the config key, i.e "metrics.reporter".
This logs something like this:
```
2016-12-09 13:40:50,287 INFO
org.apache.flink.metrics.runtime.MetricRegistry - Configuring
StatsDReporter with {port=8125, host=localhost,
class=org.apache.flink.metrics.statsd.StatsDReporter}
```
Second, we log in each reporter which properties were actually used from
the configuration. In conjunction with the above this allows users to detect
typos in specific properties, by checking which properties that were supposedly
configured but not used in the end.
This logs:
```
2016-12-09 13:40:50,287 INFO
org.apache.flink.metrics.statsd.StatsDReporter - Configured
StatsDReporter with {host:localhost, port:8125}
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zentol/flink 5307_log_rep_config
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/2979.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 #2979
----
commit 73e0eccae5e271eadd0831aefa14389cadecac8d
Author: zentol <[email protected]>
Date: 2016-12-09T12:25:11Z
[FLINK-5307] [metrics] Log reporter configuration
----
---
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.
---
