----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49014/ -----------------------------------------------------------
(Updated June 21, 2016, 7:10 p.m.) Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. Changes ------- - Changed code to deposit/set normalized value for 'tez.am.resource.memory.mb'. - Aded fix for safeguaring 'llap' Queue creation to max. size of 100%, in case the minimum size required caluclated size is more than 100%. Bugs: AMBARI-17331 https://issues.apache.org/jira/browse/AMBARI-17331 Repository: ambari Description (updated) ------- The current computations done for tez ends up with a large AM size. For LLAP, we'd like stack-advisor to change these computations. - Total Cluster Memory <=4GB - Tez AM size = 256MB and then normalized based on YARN minimum container size. - Total Cluster Memory >4GB && <= 72GB - Tez AM size = 512MB and then normalized based on YARN minimum container size. - Total Cluster Memory >72GB - Tez AM size = 1536MB and then normalized based on YARN minimum container size. Fix for safeguaring 'llap' Queue creation to max. size of 100%, in case the minimum size required caluclated size is more than 100%. Diffs (updated) ----- ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/tez-interactive-site.xml 3c83c5c ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 76654c3 ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py 1bc53ea Diff: https://reviews.apache.org/r/49014/diff/ Testing ------- Python UT added. - Python UT passes. Thanks, Swapan Shridhar