ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Levas" <rle...@hortonworks.com>
Subject Re: Review Request 32060: Add Host incorrectly changes webhcat properties
Date Sat, 14 Mar 2015 01:24:24 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32060/#review76471
-----------------------------------------------------------



ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json
<https://reviews.apache.org/r/32060/#comment124014>

    ${clusterHostInfo/hive_metastore_host} may resolved to a comma-delimited list of hosts,
so the resulting string may be 
    ```
    ...thrift://host1,host2,host3:9083,...
    ``` 
    which is probably not what you want.


- Robert Levas


On March 13, 2015, 6:20 p.m., Jaimin Jetly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32060/
> -----------------------------------------------------------
> 
> (Updated March 13, 2015, 6:20 p.m.)
> 
> 
> Review request for Ambari and Robert Levas.
> 
> 
> Bugs: AMBARI-10068
>     https://issues.apache.org/jira/browse/AMBARI-10068
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> - Singlenode cluster c6402 with hdfs, yarn,mr,hive,tez,zk,mertics, kerberos enabled.
> - After all is running fine, add host c6403 with just DN, NM and clients.
> - After add host says I need to restart hive
> - I go look at the configs that have changed, and seems Ambari modified the webhcat-site,
templeton.hive.properties.
> - It's saying the metastore moved to c6403, which it has not. It should not have modified
the webhcat-site.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json ad5ee3e

> 
> Diff: https://reviews.apache.org/r/32060/diff/
> 
> 
> Testing
> -------
> 
> tested the patch manually by going through STR on a cluster
> 
> 
> Thanks,
> 
> Jaimin Jetly
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message