-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50995/
-----------------------------------------------------------

(Updated Aug. 18, 2016, 6:03 p.m.)


Review request for Ambari, Aravindan Vijayan, Sumit Mohanty, Sid Wagle, and 
Yusaku Sako.


Changes
-------

Switched branch to trunk


Bugs: AMBARI-17591
    https://issues.apache.org/jira/browse/AMBARI-17591


Repository: ambari


Description
-------

Created metrics service which has a JVM metrics source and a sink. The JVM 
metrics source collect JVM metrics and pass it to the sink which then publish 
the metrics to AMS


Diffs (updated)
-----

  ambari-server/conf/unix/metrics.properties PRE-CREATION 
  ambari-server/pom.xml 0d91458 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
 097f01c 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
 256f0d1 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/AmbariMetricSink.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/MetricsService.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/MetricsSource.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AbstractMetricsSource.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariMetricSinkImpl.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/Configuration.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/JvmMetricsSource.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/MetricsServiceImpl.java
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/files/grafana-dashboards/default/grafana-ambari-server.json
 PRE-CREATION 

Diff: https://reviews.apache.org/r/50995/diff/


Testing
-------

Manually tested it.


Thanks,

Li-Wei Tseng

Reply via email to