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




ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
 (line 206)
<https://reviews.apache.org/r/50679/#comment210430>

    Are these two lines needed?


- Sumit Mohanty


On Aug. 1, 2016, 11:26 p.m., Swapan Shridhar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/50679/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2016, 11:26 p.m.)
> 
> 
> Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Sumit 
> Mohanty.
> 
> 
> Bugs: AMBARI-17983
>     https://issues.apache.org/jira/browse/AMBARI-17983
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> (1). Revert AMBARI-16031 (Create "/hadoop/llap/local" on each host and disk 
> in Kerberized cluster for LLAP)
>    
>    - Earlier, we had logic for using YARN work dirs for LLAP by HIVE config 
> 'hive.llap.daemon.work.dirs' having same value as of YARN config 
> 'yarn.nodemanager.local-dirs'.
>    - In kerberized setup, we were creating separate directory for LLAP, where 
> Hive user had persmissions.
>    
>    
>    - That is not required now, as HIVE is itself taking care of workdirs in 
> kerberized and un-kerberized environment. 
>    - Thus, the revert.
> 
> (2). Remove the config 'hive.llap.daemon.work.dirs' as HIVE will manage the 
> work directories itself.
> 
> 
> Diffs
> -----
> 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/metainfo.xml 
> 35329bc 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/nodemanager.py
>  106b734 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py
>  154e433 
>   
> ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py
>  cf5736e 
>   
> ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-site.xml
>  776c2b6 
>   ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/kerberos.json 
> 4fae665 
>   ambari-web/app/controllers/main/service/item.js e3df70b 
>   ambari-web/app/messages.js 62dc058 
>   ambari-web/app/models/host_component.js 91642c9 
>   ambari-web/app/views/main/service/item.js 02ac426 
>   ambari-web/test/views/main/service/item_test.js ecfa7de 
> 
> Diff: https://reviews.apache.org/r/50679/diff/
> 
> 
> Testing
> -------
> 
> Backend:
> 
> - Python UT passes.
> 
> 
> UI:
> 
> - $ npm test
>      29426 tests complete (27 seconds)
>      154 tests pending
> 
> 
> Thanks,
> 
> Swapan Shridhar
> 
>

Reply via email to