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


Ship it!




Ship It!

- Nate Cole


On April 27, 2016, 10:23 a.m., Tom Beerbower wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46740/
> -----------------------------------------------------------
> 
> (Updated April 27, 2016, 10:23 a.m.)
> 
> 
> Review request for Ambari, John Speidel and Nate Cole.
> 
> 
> Bugs: AMBARI-16089 and AMBARI-16090
>     https://issues.apache.org/jira/browse/AMBARI-16089
>     https://issues.apache.org/jira/browse/AMBARI-16090
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> After deploying Hive+Atlas via Ambari, the value of 'atlas.cluster.name‘ in 
> 'Advanced hive-site’ is ‘default’. This results in incorrect value in entity 
> notifications to Ranger.
> 
> Also, to enable login to Atlas UI, following properties have to be manually 
> added to ‘Custom application-properties’ in Atlas. 
> 
>     atlas.login.method=file
>     atlas.login.credentials.file=/etc/atlas/conf/users-credentials.properties
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py
>  c86dbf9 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/configuration/hive-site.xml
>  e56c3e9 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py
>  3d6c096 
>   ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py 
> f2e9924 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/ATLAS/configuration/application-properties.xml
>  99fd20f 
>   ambari-server/src/test/python/stacks/2.3/common/test_stack_advisor.py 
> 617f73d 
> 
> Diff: https://reviews.apache.org/r/46740/diff/
> 
> 
> Testing
> -------
> 
> New stack advisor unit test.
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Tom Beerbower
> 
>

Reply via email to