Github user ptgoetz commented on the issue:
https://github.com/apache/storm/pull/2203
@arunmahadevan
> Currently this patch addresses registering custom metrics and sending the
metrics out via reporters and does not send any metrics to nimbus right? Why is
disruptor metrics handled as a part of this?
No, this does not send any metrics to nimbus. Disruptor metrics are
included because many users would like to track those metrics (and they aren't
available via Storm UI/REST).
> There will be one thread per reporter per worker which sends out the
metrics to the destination? Just thinking if it would overwhelm the destination
and would need some local aggregation and have the supervisor report the
metrics.
The idea here was to purposely avoid attempting aggregation, since in some
cases it's impossible (e.g. there's no way to aggregate histograms). Instead,
any aggregation is assumed to take place in the destination system (Grafana,
etc.).
---
