agresch commented on a change in pull request #3170: STORM-3541 report v2 metrics api in metrics tick URL: https://github.com/apache/storm/pull/3170#discussion_r362978274
########## File path: docs/metrics_v2.md ########## @@ -145,3 +145,6 @@ public interface StormMetricsFilter extends MetricFilter { } ``` +V2 metrics can also be reported to the Metrics Consumers registered with topology.metrics.consumer.register by enabling the +topology.enable.v2.metrics.tick configuration. Review comment: I'm not clear what newline you're referring to. The one at the end of line 149? Let me know and I will put up a PR with the other changes. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services