----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60399/#review178808 -----------------------------------------------------------
Ship it! Ship It! - Sid Wagle On June 23, 2017, 7:42 p.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60399/ > ----------------------------------------------------------- > > (Updated June 23, 2017, 7:42 p.m.) > > > Review request for Ambari, Dmytro Sen, Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-21329 > https://issues.apache.org/jira/browse/AMBARI-21329 > > > Repository: ambari > > > Description > ------- > > On a cluster with HDF 3.0 and Ambari 2.5.1.0, no data is seen in Grafana > dashboards with Hosts dropdown selection. > > > Diffs > ----- > > > ambari-metrics/ambari-metrics-common/src/main/java/org/apache/hadoop/metrics2/sink/timeline/AbstractTimelineMetricsSink.java > b8cba25 > > ambari-metrics/ambari-metrics-common/src/test/java/org/apache/hadoop/metrics2/sink/timeline/cache/HandleConnectExceptionTest.java > 32fe32e > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/HBaseTimelineMetricStore.java > 18a940d > > ambari-metrics/ambari-metrics-timelineservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/metrics/timeline/TimelineMetricConfiguration.java > beb0904 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariMetricSinkImpl.java > 767398b > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/DatabaseMetricsSource.java > fa04f74 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/JvmMetricsSource.java > dbb51cf > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/MetricsServiceImpl.java > d7d905c > > > Diff: https://reviews.apache.org/r/60399/diff/1/ > > > Testing > ------- > > Manually tested. > mvn clean test passed. > > > Thanks, > > Aravindan Vijayan > >
