----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62809/#review187393 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java Lines 2782 (patched) <https://reviews.apache.org/r/62809/#comment264354> Maybe we should introduce a property updater that handles only `%HOSTGROUP::name%` constructs. ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java Lines 2783 (patched) <https://reviews.apache.org/r/62809/#comment264353> Maybe we should introduce a new property updater for this property that handles only `%HOSTGROUP::name%` constructs. ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java Lines 2784 (patched) <https://reviews.apache.org/r/62809/#comment264352> Why property updater for `druid.zk.service.host` is not needed any more? - Sebastian Toader On Oct. 9, 2017, 9:16 a.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62809/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2017, 9:16 a.m.) > > > Review request for Ambari, Nishant Bangarwa, Slim Bouguerra, Sandor Magyari, > Swapan Shridhar, and Sebastian Toader. > > > 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 > ----- > > > 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/1/ > > > Testing > ------- > > Tested manually on local cluster. > > Unit tests pass: `Tests run: 4900, Failures: 0, Errors: 0, Skipped: 36` > > > Thanks, > > Attila Doroszlai > >