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


Ship it!




Ship It!

- Alejandro Fernandez


On June 2, 2016, 10:32 p.m., Swapan Shridhar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48178/
> -----------------------------------------------------------
> 
> (Updated June 2, 2016, 10:32 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez and Sumit Mohanty.
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> **- AMBARI-17017. Fixing open issues for 'Hive Server Interactive' in 
> Kerberized environment.**
> 
>  As part of this, did the following :
>   - Tested YARN installation in kerberized environment after adding 
> conditional HIVE identities in YARN/kerberos.json. By conditional, 
>     it means that HIVE referenced identity will be processed only if HIVE 
> exists as a service.
>   - Added HIVE identities in YARN/kerberso.json to be installed on the 
> NOIDEMANAGER(s).
>   
> **- AMBARI-17020. Use 'llapstatus' comand after starting llap to check its 
> status before starting HiveServer2.**
>   
>   As part of this, did the following :
>     - Right now, after starting llap app, we wait for 30 secs before starting 
> hive2/HiveServer2.
>     - Checks 'llap' app status deployment via 'llapstatus' command and start 
> hive2/HiveServer2 only if llap app deployment was a success.
> 
>       eg: llapstatus command: hive2/bin/hive --service llapstatus --name 
> <llap app name>
>       
> **Further, following issues have been fixed as part of this patch:**
> 
> - Kerberos enabled cluster does not work with Tez as 
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json
>  has :
>   "hive.llap.daemon.work.dirs": "/hadoop/llap/local". This should be only in 
> 2.5/HIVE, where it already exists.
>   
>   
> - Cleaned the TODO comments in code pertaining to HSI.
>  
> 
> - Removed the addition of args "-XX:MaxDirectMemorySize" by Ambari during 
> LLAP package creation, as HIVE driver is taking care of it by itself.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/kerberos.json
>  a9e3840 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_interactive.py
>  14ba0f9 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py
>  77e1cd4 
>   
> ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_service_interactive.py
>  908c726 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/YARN/kerberos.json 
> e22b595 
> 
> Diff: https://reviews.apache.org/r/48178/diff/
> 
> 
> Testing
> -------
> 
> - Python UT passes.
> 
> 
> Thanks,
> 
> Swapan Shridhar
> 
>

Reply via email to