----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58990/#review174360 -----------------------------------------------------------
Ship it! Ship It! - Robert Nettleton On May 9, 2017, 9:38 a.m., Laszlo Puskas wrote: > > ----------------------------------------------------------- > 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 > ------- > > 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 > ------- > > Tested successfully on local environment. > Added unit tests. > > Unit tests - success. > > > Thanks, > > Laszlo Puskas > >