----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36473/ -----------------------------------------------------------
(Updated July 22, 2015, 5 p.m.) Review request for samza. Repository: samza Description ------- SAMZA-733 Add metrics to Elasticsearch System Producer Diffs (updated) ----- samza-core/src/main/java/org/apache/samza/metrics/MetricGroup.java PRE-CREATION samza-core/src/main/java/org/apache/samza/metrics/MetricsBase.java PRE-CREATION samza-core/src/main/scala/org/apache/samza/metrics/MetricsHelper.scala 8eac8ef samza-elasticsearch/src/main/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemFactory.java a277b69 samza-elasticsearch/src/main/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemProducer.java 7eb14a2 samza-elasticsearch/src/main/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemProducerMetrics.java PRE-CREATION samza-elasticsearch/src/test/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemProducerMetricsTest.java PRE-CREATION samza-elasticsearch/src/test/java/org/apache/samza/system/elasticsearch/ElasticsearchSystemProducerTest.java e63d62c Diff: https://reviews.apache.org/r/36473/diff/ Testing ------- Tested that metrics for Elasticsearch producer appear in JMX and the metrics stream and that the metrics correctly count how many Elasticsearch documents were created and indexed. Thanks, Roger Hoover
