> On April 19, 2016, 6:47 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py, > > line 306 > > <https://reviews.apache.org/r/46358/diff/2/?file=1349441#file1349441line306> > > > > Prefix the function with "_" since scope of this function is very > > limited to LLAP.
Fixed. > On April 19, 2016, 6:47 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py, > > line 312 > > <https://reviews.apache.org/r/46358/diff/2/?file=1349441#file1349441line312> > > > > Should check that output is not None first. > > Perhaps include in the code comments what the sample output looks like. Now checking it in receiver. > On April 19, 2016, 6:47 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py, > > line 389 > > <https://reviews.apache.org/r/46358/diff/2/?file=1349441#file1349441line389> > > > > Since this is a while True loop, I prefer to use >= here, and > > "if app_running_partial_wait_ctr < num_times_to_iterate" above Code not more there. - Swapan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46358/#review129581 ----------------------------------------------------------- On April 19, 2016, 10:41 p.m., Swapan Shridhar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46358/ > ----------------------------------------------------------- > > (Updated April 19, 2016, 10:41 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Sumit Mohanty. > > > Bugs: AMBARI-15959 > https://issues.apache.org/jira/browse/AMBARI-15959 > > > Repository: ambari > > > Description > ------- > > Incorporate 'llap status check' to be used for (1). Before Hive Server > Interactive start, and (2). status call. > > - Further, changed hive.llap.daemon.service.hosts value to @llap. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py > 9d011a0 > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/params_linux.py > 9bee786 > > ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-env.xml > 88541bd > > ambari-server/src/main/resources/stacks/HDP/2.5/services/HIVE/configuration/hive-interactive-site.xml > 0b3de15 > ambari-server/src/test/python/stacks/2.5/HIVE/appComplete.json PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/invalidApp.json PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/oneContainerDown.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/oneContainerDown1.json > PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/running.json PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/starting.json PRE-CREATION > ambari-server/src/test/python/stacks/2.5/HIVE/test_hive_server_int.py > ac4f6fc > ambari-server/src/test/python/stacks/2.5/configs/hsi_default.json f9c31dc > > Diff: https://reviews.apache.org/r/46358/diff/ > > > Testing > ------- > > - Python UT passes. > > > Thanks, > > Swapan Shridhar > >
