[GitHub] flink issue #5857: [FLINK-9187][METRICS] add prometheus pushgateway reporter

[lamber-ken]

Github user lamber-ken commented on the issue:

    https://github.com/apache/flink/pull/5857
  
    @zentol, @tillrohrmann, cc
    
    I don't think there is a relationship between `FLINK-9543` and `FLINK-9187`.
    At present, the focus of our discussion is on the job name of pushgateway.
    My point is that there is no need to use JMID or TMID to form the job name 
of pushgateway, because the job name of pushgateway is just used to distinguish 
different services(like TM or JM and), and prevent the same metrics from being 
covered, it doesn't make sense for metrics data.
    
    A minimal reproducible example, two flink cluster, each cluster includes 
one JM and three TMs.
    
![image](https://user-images.githubusercontent.com/20113411/41583819-31d803bc-73d8-11e8-8d76-fecc9801638c.png)
    
![image](https://user-images.githubusercontent.com/20113411/41584075-e7a176e2-73d8-11e8-8174-dab68f4088ed.png)
    
![image](https://user-images.githubusercontent.com/20113411/41584093-f70c85e0-73d8-11e8-8aa8-46745db879aa.png)
    
    
    
    



---