> On Jan. 6, 2017, 9:23 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariPerformanceMonitor.java, > > line 75 > > <https://reviews.apache.org/r/55278/diff/1/?file=1598877#file1598877line75> > > > > Why not drop this, the vars are unambiguous anyways.
Done. > On Jan. 6, 2017, 9:23 p.m., Sid Wagle wrote: > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/MetricsServiceImpl.java, > > line 109 > > <https://reviews.apache.org/r/55278/diff/1/?file=1598881#file1598881line109> > > > > formatting Done. - Aravindan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55278/#review160764 ----------------------------------------------------------- On Jan. 6, 2017, 10:44 p.m., Aravindan Vijayan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55278/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2017, 10:44 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Dmytro Sen, Jonathan Hurley, > Sumit Mohanty, and Sid Wagle. > > > Bugs: AMBARI-17596 > https://issues.apache.org/jira/browse/AMBARI-17596 > > > Repository: ambari > > > Description > ------- > > Extend the eclipse link PeformanceMonitor utility to collect and send metrics > to AMS. > Counter, Timer and Avg time (Timer/Counter) metrics are tracked. > > The metrics.properties file has enough information to know how to configure > the Database metrics source to collect metrics of interest. > > > Diffs > ----- > > ambari-server/conf/unix/metrics.properties 3ee22d6 > ambari-server/conf/windows/metrics.properties 3ee22d6 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/MetricsService.java > 4a613f0 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/MetricsSource.java > 400dcb6 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AbstractMetricsSource.java > 6bdd0ba > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariMetricSinkImpl.java > be24988 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/AmbariPerformanceMonitor.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/DatabaseMetricsSource.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/JvmMetricsSource.java > cb9f275 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/MetricsConfiguration.java > ca83a53 > > ambari-server/src/main/java/org/apache/ambari/server/metrics/system/impl/MetricsServiceImpl.java > d0d2e69 > > ambari-server/src/test/java/org/apache/ambari/server/metric/system/impl/JvmMetricsSourceTest.java > 9f649b4 > > ambari-server/src/test/java/org/apache/ambari/server/metric/system/impl/MetricsServiceTest.java > 4029f25 > > ambari-server/src/test/java/org/apache/ambari/server/metric/system/impl/MetricsSourceTest.java > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/metric/system/impl/TestAmbariMetricsSinkImpl.java > 3565504 > > ambari-server/src/test/java/org/apache/ambari/server/metric/system/impl/TestMetricsSource.java > acf1586 > ambari-server/src/test/resources/metrics.properties 5eee064 > > Diff: https://reviews.apache.org/r/55278/diff/ > > > Testing > ------- > > Manually tested. > Unit tests added. > > > File Attachments > ---------------- > > Sample Grafana dashboard for AmbariServer DB metrics > > https://reviews.apache.org/media/uploaded/files/2017/01/06/a6f88cb6-743f-44ed-a851-3e959072b176__Screen_Shot_2017-01-06_at_12.59.00_PM.png > > > Thanks, > > Aravindan Vijayan > >
