> On Oct. 6, 2017, 9:47 p.m., Nishant Bangarwa wrote:
> > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessor.java
> > Lines 2782 (patched)
> > <https://reviews.apache.org/r/62809/diff/1/?file=1847507#file1847507line2782>
> >
> > this seems odd, metadata store is not always installed on the same host
> > as coordinator.
> > Fwiw, metadata store for druid is not installed/managed by ambari, user
> > needs to set it up and provide valid hostname and uri.
For explicit `%HOSTGROUP::some_host_group%` the hostname from the given host
group is substituted, component is ignored.
Component ("DRUID_COORDINATOR") is relevant only if property value contains
"localhost".
- Attila
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62809/#review187297
-----------------------------------------------------------
On Oct. 6, 2017, 5:35 p.m., Attila Doroszlai wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62809/
> -----------------------------------------------------------
>
> (Updated Oct. 6, 2017, 5:35 p.m.)
>
>
> Review request for Ambari, Nishant Bangarwa, Slim Bouguerra, and Swapan
> Shridhar.
>
>
> 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
>
>
