[GitHub] flink issue #3833: [FLINK-6221] Add PrometheusReporter

2017-05-11 Thread hadronzoo
Github user hadronzoo commented on the issue: https://github.com/apache/flink/pull/3833 @mbode thanks for working on this! One thing that I've found useful when exporting Flink's statsd metrics to Prometheus is to make several of the metric fields tags: like

[GitHub] flink issue #3576: [FLINK-6132]Remove redundant code in CliFrontend.java

2017-03-21 Thread hadronzoo
Github user hadronzoo commented on the issue: https://github.com/apache/flink/pull/3576 LGTM @zhengcanbin --- 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

[GitHub] flink pull request #3538: FLINK-6051: Correct metrics scope names

2017-03-14 Thread hadronzoo
GitHub user hadronzoo opened a pull request: https://github.com/apache/flink/pull/3538 FLINK-6051: Correct metrics scope names Closes FLINK-6051. You can merge this pull request into a Git repository by running: $ git pull https://github.com/orgsync/flink FLINK-6051