----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39041/ -----------------------------------------------------------
Review request for samza. Repository: samza Description ------- Added factory and reporter class for CSV metrics. Notice that SamzaCSVReporter uses MetricsRegistryReporterMap introduced in [SAMZA-338](https://issues.apache.org/jira/browse/SAMZA-338) and CSVReporterWriter extends ReporterWriter also introduced in mentioned patch. This patch is dependent on [SAMZA-338](https://issues.apache.org/jira/browse/SAMZA-338) and [SAMZA-683](https://issues.apache.org/jira/browse/SAMZA-683). Documentation for this metrics is not included yet, because SAMZA-338, SAMZA-339, SAMZA-340 and SAMZA-341 modify same files: metrics.md, configuration-table.html. Diffs ----- samza-core/src/main/java/org/apache/samza/metrics/reporter/CSVReporterFactory.java PRE-CREATION samza-core/src/main/java/org/apache/samza/metrics/reporter/CSVReporterWriter.java PRE-CREATION samza-core/src/main/java/org/apache/samza/metrics/reporter/SamzaCSVReporter.java PRE-CREATION Diff: https://reviews.apache.org/r/39041/diff/ Testing ------- Testing done with hello-samza. Thanks, Aleksandar Pejakovic
