> On May 14, 2015, 7:09 a.m., Luis De Pombo wrote: > > Thanks for the comments Yan! I will address them and will follow up on the > > issue around org.apache.samza.metrics.Snapshot.getMax()
I will also add to the documentation! - Luis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34011/#review83727 ----------------------------------------------------------- On May 11, 2015, 10:10 a.m., Luis De Pombo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34011/ > ----------------------------------------------------------- > > (Updated May 11, 2015, 10:10 a.m.) > > > Review request for samza. > > > Repository: samza > > > Description > ------- > > Add support for a Graphite Metrics Reporter > > > Diffs > ----- > > build.gradle ac80a8664180e556ec83e229e04e3d8c56b70506 > checkstyle/import-control.xml 5f8e103a2e43f96518b20de1c7cbd84e0af24842 > gradle/dependency-versions.gradle ee6dfc411b7ab90b187df79f109884127953862e > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/GraphiteCounter.java > PRE-CREATION > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/GraphiteGauge.java > PRE-CREATION > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/GraphiteReporterFactory.java > PRE-CREATION > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/GraphiteSnapshot.java > PRE-CREATION > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/GraphiteTimer.java > PRE-CREATION > > samza-graphite/src/main/java/org/apache/samza/metrics/reporter/SamzaGraphiteReporter.java > PRE-CREATION > > samza-graphite/src/test/java/org/apache/samza/metrics/reporter/GraphiteCounterTest.java > PRE-CREATION > > samza-graphite/src/test/java/org/apache/samza/metrics/reporter/GraphiteGaugeTest.java > PRE-CREATION > > samza-graphite/src/test/java/org/apache/samza/metrics/reporter/GraphiteSnapshotTest.java > PRE-CREATION > > samza-graphite/src/test/java/org/apache/samza/metrics/reporter/GraphiteTimerTest.java > PRE-CREATION > > samza-graphite/src/test/java/org/apache/samza/metrics/reporter/SamzaGraphiteReporterTest.java > PRE-CREATION > settings.gradle bb07a3b84b14dcef94da1bb166eab6aa3d0026bb > > Diff: https://reviews.apache.org/r/34011/diff/ > > > Testing > ------- > > > Thanks, > > Luis De Pombo > >