[STORM-3670] Separate configurations for daemon metric reporters and topology metrics reporters

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.3.0
• Assignee: Ethan Li
• Reporter: Ethan Li
• Created: 08/Jul/20 18:39
• Updated: 06/Aug/20 15:01
• Resolved: 06/Aug/20 14:55

Description

1. storm.metrics.reporters should be configurable per topology.
Currently, storm.metrics.reporters config is retrieved from storm daemon config (more precisely, supervisor config) so it can't be configured at topology level. User should be able to choose their own reporter for their topology

2. storm.metrics.reporters is actually only used in workers. So we should remove unnecessary "daemons" section in the reporter conf to make it clear

3. configs like storm.daemon.metrics.reporter.plugin.rate.unit should only be used by daemon metric reporters (storm.daemon.metrics.reporter.plugins)

Issue Links

• is related to: STORM-3688 Copy storm.daemon.metrics.reporter.plugin.* configs from Config class to DaemonConfig class and deprecate them in Config class