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

(Updated March 21, 2016, 6:52 p.m.)


Review request for Ambari, Myroslav Papirkovskyy, Robert Levas, and Sandor 
Magyari.


Changes
-------

Use cluster.getServiceByConfigType instead of hardcoded config type.


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


Repository: ambari


Description
-------

The logic that determines if a new version has to be created for a 
configuration should skip cluster wide config types when verifying current 
desired config against service configs as these are not service configs.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 ac2fb22 
  ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java 
80697f1 

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


Testing
-------

Manual testing done.

Unit tests results:
Results :

Tests run: 3534, Failures: 0, Errors: 0, Skipped: 36


Thanks,

Sebastian Toader

Reply via email to