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

(Updated Aug. 1, 2016, 10:11 p.m.)


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


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


Repository: ambari


Description
-------

Added new Interface to drain aggregated metrics to external storage. Added unit 
test and sample implementation (TimelineMetricsAggregatorMemorySink.java) to 
store metrics in memory.


Diffs (updated)
-----

  
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessor.java
 dfd730d 
  
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricConfiguration.java
 e82d65a 
  
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorSink.java
 PRE-CREATION 
  
ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessorTest.java
 0ea668a 
  
ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricsAggregatorMemorySink.java
 PRE-CREATION 

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


Testing
-------

Manually tested.


Added unit test.


mvn clean test


Thanks,

Jameel Mohamed

Reply via email to