-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58990/
-----------------------------------------------------------
(Updated May 9, 2017, 9:38 a.m.)
Review request for Ambari, Attila Doroszlai, Robert Levas, Robert Nettleton,
and Sandor Magyari.
Bugs: AMBARI-20872
https://issues.apache.org/jira/browse/AMBARI-20872
Repository: ambari
Description (updated)
-------
Validating required properties for services listed in the blueprint can't be
accurate as further configurtion can be added in the cluster creation template.
(Where the configuration is added varies by use cases and usage)
This patch contains the change that defers this validation till all the
configuration is together, namely till the cluster creation template is posted.
The validation logic is the same as it was before.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/topology/BlueprintValidatorImpl.java
45a8c5c
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyValidator.java
146b424
ambari-server/src/main/java/org/apache/ambari/server/topology/validators/RequiredConfigPropertiesValidator.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/topology/validators/TopologyValidatorFactory.java
0e77301
ambari-server/src/test/java/org/apache/ambari/server/topology/BlueprintImplTest.java
0608697
ambari-server/src/test/java/org/apache/ambari/server/topology/validators/RequiredConfigPropertiesValidatorTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/58990/diff/2/
Testing (updated)
-------
Tested successfully on local environment.
Added unit tests.
Unit tests - success.
Thanks,
Laszlo Puskas
