----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62809/ -----------------------------------------------------------
(Updated Oct. 9, 2017, 2:43 p.m.) Review request for Ambari, Nishant Bangarwa, Slim Bouguerra, Sandor Magyari, Swapan Shridhar, and Sebastian Toader. Changes ------- extracted property updater that only replaces `%HOSTGROUP::name%` placeholders Bugs: AMBARI-22159 https://issues.apache.org/jira/browse/AMBARI-22159 Repository: ambari Description ------- Replace placeholders in `metastore_hostname`, `druid.metadata.storage.connector.connectURI` and `druid.zk.service.host` when creating cluster via blueprint. Diffs (updated) ----- ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java 5a6e2cc0de7dc0c0f4499624e21c43bbc8d24b54 ambari-server/src/test/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java 68d6349db0207ab6df44b4b0a341c77ecabc46ea Diff: https://reviews.apache.org/r/62809/diff/2/ Changes: https://reviews.apache.org/r/62809/diff/1-2/ Testing ------- Tested manually on local cluster. Unit tests pass: `Tests run: 4900, Failures: 0, Errors: 0, Skipped: 36` Thanks, Attila Doroszlai