samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Shanthoosh Venkataraman <>
Subject Review Request 53002: Do not load the monitor, if the MonitorFactoryClass is not defined for the monitor in the config.
Date Tue, 18 Oct 2016 22:53:52 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for samza and Jake Maes.

Repository: samza


This patch aims to not load the monitors if the monitor factory class is not defined(set)
for the monitor in the config. This will enable the users to turn on/off the monitors in samza-rest
easily(just by setting the monitorFactoryClass config associated with monitor a to empty string.)


  samza-rest/src/main/java/org/apache/samza/monitor/ ce947f7ae1175acc1ee9aa75991c726848072694

  samza-rest/src/test/java/org/apache/samza/monitor/ 4618b54f5af861383df45bf7185622d36d17cd5e

  samza-rest/src/test/java/org/apache/samza/monitor/mock/ PRE-CREATION



Unit testing and manual testing are done to verify the functionality.


Shanthoosh Venkataraman

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