-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38105/
-----------------------------------------------------------

(Updated Sept. 11, 2015, 3:09 p.m.)


Review request for Falcon.


Bugs: FALCON-1425
    https://issues.apache.org/jira/browse/FALCON-1425


Repository: falcon-git


Description
-------

Provide Email based notification plugin to send notification when Falcon 
instance completes.


Diffs (updated)
-----

  client/src/main/java/org/apache/falcon/entity/v0/EntityNotification.java 
PRE-CREATION 
  client/src/main/resources/feed-0.1.xsd 4ff8baa 
  client/src/main/resources/jaxb-binding.xjb f644f40 
  client/src/main/resources/process-0.1.xsd c81d6f7 
  common/src/main/java/org/apache/falcon/entity/EntityUtil.java 25d9008 
  common/src/main/resources/startup.properties c48188c 
  metrics/pom.xml 3d558fc 
  metrics/src/main/java/org/apache/falcon/plugin/NotificationPlugin.java 
PRE-CREATION 
  metrics/src/main/java/org/apache/falcon/util/EmailNotificationArgs.java 
PRE-CREATION 
  metrics/src/main/java/org/apache/falcon/util/NotificationType.java 
PRE-CREATION 
  
oozie/src/test/java/org/apache/falcon/oozie/feed/OozieFeedWorkflowBuilderTest.java
 2f7787d 
  
oozie/src/test/java/org/apache/falcon/oozie/process/OozieProcessWorkflowBuilderTest.java
 3aaf304 
  oozie/src/test/resources/config/process/process-notification.xml PRE-CREATION 
  oozie/src/test/resources/feed/feed-notification.xml PRE-CREATION 
  prism/pom.xml 52b558d 
  prism/src/main/java/org/apache/falcon/plugin/EmailNotification.java 
PRE-CREATION 
  prism/src/main/java/org/apache/falcon/plugin/EmailNotificationPlugin.java 
PRE-CREATION 
  prism/src/main/java/org/apache/falcon/plugin/NotificationHandler.java 
PRE-CREATION 
  prism/src/main/java/org/apache/falcon/util/NotificationUtil.java PRE-CREATION 
  prism/src/test/java/org/apache/falcon/plugin/EmailNotificationTest.java 
PRE-CREATION 
  src/conf/startup.properties 9925373 

Diff: https://reviews.apache.org/r/38105/diff/


Testing
-------

Yes, manual testing has been done for this after configuring startup.properties 
with SMTP properties. 
Also test cases has been added to test Falcon feed/process entity with 
notification tag.
Unit test has been added to test Email Notification.


Thanks,

Peeyush Bishnoi

Reply via email to