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

Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Sumit Mohanty.


Repository: ambari


Description
-------

- Added 'args' to LLAP package creation. Specific parameter related to cache 
added.
- disabling call to llap status as its not working correctly.
- Detect and calculate cache config 'hive.llap.io.memory.size' only if user has 
not supplied the value. Otherwise use the value supplied by user.
- Added 'hive-interactive-site' config 'hive.llap.io.memory.mode' = 'none' and 
'hive.llap.io.enabled'. Cache config 'hive.llap.io.memory.size' value >= 64m 
implies config 'hive.llap.io.enabled' set to true, else false.
- Removed the "hive.tez.container.size" from hive-interactive-site.xml and 
using it from hive-site.xml, as both Hive and Hive2 are supposed to use same 
calulated value.


Diffs
-----

  
ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py
 882136a 
  
ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-site.xml
 3f110dc 
  ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 
ca92157 
  ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 574964f 

Diff: https://reviews.apache.org/r/46877/diff/


Testing
-------

Yes.
Python UT passes.


Thanks,

Swapan Shridhar

Reply via email to