samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksandar Pejakovic" <a.pejako...@levi9.com>
Subject Review Request 39041: SAMZA-339 Support metrics report via CSV
Date Tue, 06 Oct 2015 08:01:41 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message