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

(Updated March 23, 2017, 1:51 p.m.)


Review request for Ambari, Attila Doroszlai, Robert Levas, and Sebastian Toader.


Changes
-------

Fixed review notes.


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


Repository: ambari


Description
-------

Problem:
When validating config types posted in the blueprint and cluster template need 
to be checked against config types for services in the blueprint available in 
the stack.
Previously only config types in the cluster template were considered during the 
validation.

Solution
All the config types are taken into account during the above validation, by 
calling the appropriate internal api.

Note:
A bug has also been corrected in finding valid/invalid config types.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/topology/validators/ClusterConfigTypeValidator.java
 18d08b9 
  
ambari-server/src/test/java/org/apache/ambari/server/topology/validators/ClusterConfigTypeValidatorTest.java
 24fa8b8 


Diff: https://reviews.apache.org/r/57874/diff/2/

Changes: https://reviews.apache.org/r/57874/diff/1-2/


Testing
-------

Manually on local env.
Unit tests written.


Thanks,

Laszlo Puskas

Reply via email to