----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46183/ -----------------------------------------------------------
(Updated April 15, 2016, 10:29 p.m.) Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Sumit Mohanty. Changes ------- Alejandro's suggestion incorporated. Bugs: AMBARI-15876 https://issues.apache.org/jira/browse/AMBARI-15876 Repository: ambari Description ------- -- When Hive Server Interactive is turned ON: Ambari creates OR "modifies 'llap' queue 'state and/or capacity' " based on below conditions: - if only 1 queue exists at root level and is 'default' queue and has 100% cap -> Creates 'llap' queue, OR - if 2 queues exists at root level ('llap' and 'default') : - 'llap' queue state is STOPPED -> Modifies 'llap' queue state to RUNNING, adjusts capacity, OR - 'llap' queue state is RUNNING and 'llap_queue_capacity' prop != 'llap' queue current running capacity -> Modifies 'llap' queue capacity to 'llap_queue_capacity'. -- When Hive Server Interactive is turned OFF: - Checks and sees (1). If only two leaf queues exist at root level, namely: 'default' and 'llap', and (2). 'llap' is in RUNNING state. If yes, performs the following actions: (1). 'llap' queue state set to STOPPED, (2). 'llap' queue capacity set to 0 %, (3). 'default' queue capacity set to 100 % Diffs (updated) ----- ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-env.xml d518c8b ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-site.xml ba46f00 ambari-server/src/main/resources/stacks/HDP/2.5/services/YARN/configuration/capacity-scheduler.xml PRE-CREATION ambari-server/src/main/resources/stacks/HDP/2.5/services/stack_advisor.py 7407da3 ambari-server/src/test/python/stacks/2.5/common/test_stack_advisor.py c21bad7 Diff: https://reviews.apache.org/r/46183/diff/ Testing ------- - Python UT passes. - Manual testing done. Various scenarios tested. - Test cases added to test the creation and disabling logic. - Screenshots attached. File Attachments ---------------- LLAP queue created https://reviews.apache.org/media/uploaded/files/2016/04/14/4304fa89-fcfa-47b2-9240-d4ffa221cb76__Screen_Shot_2016-04-13_at_4.51.35_PM.png HSI On and llap and default queues shown in dropdown https://reviews.apache.org/media/uploaded/files/2016/04/14/622db04f-ec39-4a63-920d-dffc3a269917__Screen_Shot_2016-04-13_at_4.51.03_PM.png YARn queue manager with llap and default queues https://reviews.apache.org/media/uploaded/files/2016/04/14/29ee3e5c-9c49-4e73-be39-46e6ec786d21__Screen_Shot_2016-04-13_at_4.48.46_PM.png Thanks, Swapan Shridhar