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

(Updated May 28, 2016, 5:15 a.m.)


Review request for Ambari, Srimanth Gunturi and Tom Beerbower.


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


Repository: ambari


Description
-------

The commit, 
https://github.com/apache/ambari/commit/89fd30b40f3108bfcbcc73eb2d74c94a2ba14a7a
 which accidentally made the storm.topology.submission.notifier.plugin property 
mandatory. If the storm defaults yaml config does not set the property a " " 
value is set for the property. This causes storm startup to fail with a Class 
Not Found exception when starting with Ambari. This change fixes the stack 
advisor to not set the property when no value is specified for it through 
command line or storm defaults.


Diffs
-----

  ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py 
79314f5 

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


Testing (updated)
-------

mvn test


Thanks,

Sivaguru Chendamaraikannan

Reply via email to