-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57874/
-----------------------------------------------------------
(Updated March 27, 2017, 11:07 a.m.)
Review request for Ambari, Attila Doroszlai, Robert Levas, and Sebastian Toader.
Changes
-------
Updated / fixed unit tests, removed unnecessary code.
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
305c88e
ambari-server/src/test/java/org/apache/ambari/server/topology/validators/ClusterConfigTypeValidatorTest.java
bb10df9
Diff: https://reviews.apache.org/r/57874/diff/4/
Changes: https://reviews.apache.org/r/57874/diff/3-4/
Testing
-------
Manually on local env.
Unit tests written.
Thanks,
Laszlo Puskas
