----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48178/#review136007 -----------------------------------------------------------
Ship it! Ship It! - Sumit Mohanty 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 > >