-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39907/
-----------------------------------------------------------
Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle.
Bugs: AMBARI-13701
https://issues.apache.org/jira/browse/AMBARI-13701
Repository: ambari
Description
-------
PURPOSE
Currently, AMS has aggregators and aggregated tables only for SECONDS, HOURS
and DAYS. We need to introduce the MINUTE aggregator and the corresponding
table to return MINUTE granularity data for appropriate time ranges.
FIX
Added the minute aggregator and changed the current 30 second aggregator to be
called as the "SECOND" aggregator.
Diffs
-----
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java
17df629
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessor.java
3ce30fd
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricConfiguration.java
30e42f2
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricAggregatorFactory.java
f07918c
ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/query/PhoenixTransactSQL.java
092c983
ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITClusterAggregator.java
cbf0233
ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITPhoenixHBaseAccessor.java
89fee7c
ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/cache/TimelineMetricCacheEntryFactory.java
7c7db9f
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog213.java
8f1a481
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-site.xml
89b584b
Diff: https://reviews.apache.org/r/39907/diff/
Testing
-------
Relevant unit tests pass.
Manual testing done.
Added new unit test to cover Minute aggregator.
Thanks,
Aravindan Vijayan
