[ 
https://issues.apache.org/jira/browse/STORM-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15253792#comment-15253792
 ] 

ASF GitHub Bot commented on STORM-1723:
---------------------------------------

Github user unsleepy22 commented on the pull request:

    https://github.com/apache/storm/pull/1352#issuecomment-213391769
  
    for the ClusterInfo class, do we need to add a "cluster.name" property? In 
a real metrics consumer case, the external storage may want to consumer a bunch 
of cluster metrics and display them, thus a "cluster.name" may be necessary to 
distinguish different clusters.


> Introduce ClusterMetricsConsumer
> --------------------------------
>
>                 Key: STORM-1723
>                 URL: https://issues.apache.org/jira/browse/STORM-1723
>             Project: Apache Storm
>          Issue Type: New Feature
>          Components: storm-core
>    Affects Versions: 2.0.0, 1.0.1
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>
> NOTE: This issue is already discussed shortly. Please refer 
> [here|http://mail-archives.apache.org/mod_mbox/storm-dev/201604.mbox/%3CCAF5108hDCcMKxLXKUYLReOoKkNNdgW2YudweR+mKr=1hlsl...@mail.gmail.com%3E]
>  for details.
> This issue focuses to introduce ClusterMetricsConsumer and provide interface 
> to let users plugin their consumers.
> ClusterMetricsConsumers will be attached to Nimbus, and leader of Nimbus will 
> push cluster related metrics to ClusterMetricsConsumer.
> Requirements of ClusterMetricsConsumer are here:
> - Only leader of Nimbus should publish cluster metrics to consumer.
> - Nimbus shouldn't be affected by crashing or heavy latency on consumer.
> - Consumer should have resilient when crashing or Nimbus should take care of.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to