[STORM-3654] JCQueue shouldn't include executor string in the metric name

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.3.0
• Component/s: storm-client
• Assignee: Rui Li
• Reporter: Rui Li
• Created: 19/Jun/20 21:44
• Updated: 01/Jul/20 14:04
• Resolved: 29/Jun/20 14:40

Description

I noticed that the JCQueue metrics have names like

receive-queue_65__65_-capacity
receive-queue_65__65_-population
receive-queue_100__100_-capacity
receive-queue_100__100_-population
..

This is not suitable for aggregation. We shouldn't include executors in the metric name.

https://github.com/apache/storm/blob/master/storm-client/src/jvm/org/apache/storm/daemon/worker/WorkerState.java#L682