[ 
https://issues.apache.org/jira/browse/STORM-3654?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ethan Li resolved STORM-3654.
-----------------------------
    Fix Version/s: 2.3.0
       Resolution: Fixed

Thanks [~li530] Merged to master

> JCQueue shouldn't include executor string in the metric name
> ------------------------------------------------------------
>
>                 Key: STORM-3654
>                 URL: https://issues.apache.org/jira/browse/STORM-3654
>             Project: Apache Storm
>          Issue Type: Improvement
>            Reporter: Rui Li
>            Assignee: Rui Li
>            Priority: Minor
>             Fix For: 2.3.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> I noticed that the JCQueue metrics have names like
> {code:java}
> receive-queue_65__65_-capacity
> receive-queue_65__65_-population
> receive-queue_100__100_-capacity
> receive-queue_100__100_-population
> ..
> {code}
> 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]
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to