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

(Updated 六月 2, 2017, 1:31 a.m.)


Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, and 
Tim Thorpe.


Bugs: AMBARI-21122
    https://issues.apache.org/jira/browse/AMBARI-21122


Repository: ambari


Description
-------

This patch will support using property ambari.dispatch-property.script.filename 
in alert target to tell AlertScriptDispatcher to lookup script by 
filename,default in  directory /var/lib/ambari-server/resources/scripts. 

We can also change this directory in ambari.properties by 
notification.dispatch.alert.script.directory property.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java
 114046f 
  
ambari-server/src/main/java/org/apache/ambari/server/notifications/dispatchers/AlertScriptDispatcher.java
 84bfe52 
  
ambari-server/src/test/java/org/apache/ambari/server/notifications/dispatchers/AlertScriptDispatcherTest.java
 f1f320d 


Diff: https://reviews.apache.org/r/59440/diff/7/

Changes: https://reviews.apache.org/r/59440/diff/6-7/


Testing
-------

Tested in a cluster


Thanks,

yao lei

Reply via email to