----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33958/#review83026 -----------------------------------------------------------
Ship it! Ship It! - Myroslav Papirkovskyy On Травень 8, 2015, 8:04 після полудня, Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33958/ > ----------------------------------------------------------- > > (Updated Травень 8, 2015, 8:04 після полудня) > > > Review request for Ambari, Dmytro Sen, Jaimin Jetly, Mahadev Konar, Myroslav > Papirkovskyy, and Sumit Mohanty. > > > Bugs: AMBARI-11010 > https://issues.apache.org/jira/browse/AMBARI-11010 > > > Repository: ambari > > > Description > ------- > > Also any new number or gauge widget created with an aggregator function by > user will not work. > > Includes some refactoring and optimization code as well. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-common/src/main/java/org/apache/hadoop/metrics2/sink/timeline/SingleValuedTimelineMetric.java > PRE-CREATION > > ambari-metrics/ambari-metrics-common/src/main/java/org/apache/hadoop/metrics2/sink/timeline/TimelineMetrics.java > 3eb0e89 > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java > 447f6f9 > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/PhoenixHBaseAccessor.java > 7258cad > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricHostAggregator.java > 18e5f18 > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/aggregators/TimelineMetricReadHelper.java > 25f8c62 > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITClusterAggregator.java > aa276f3 > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITMetricAggregator.java > f4f2223 > > ambari-metrics/ambari-metrics-timelineservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/ITPhoenixHBaseAccessor.java > 4c56f77 > > Diff: https://reviews.apache.org/r/33958/diff/ > > > Testing > ------- > > All unit tests pass. > > > Thanks, > > Sid Wagle > >
