----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45833/#review127586 -----------------------------------------------------------
Ship it! Ship It! - Andrew Onischuk On April 7, 2016, 12:55 p.m., Sebastian Toader wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45833/ > ----------------------------------------------------------- > > (Updated April 7, 2016, 12:55 p.m.) > > > Review request for Ambari, Andrew Onischuk and Dmytro Sen. > > > Bugs: AMBARI-15744 > https://issues.apache.org/jira/browse/AMBARI-15744 > > > Repository: ambari > > > Description > ------- > > Ambari uses `/usr/hdp/current/hive-webhcat/sbin/webhcat_server.sh stop` to > stop WebHCat server. Since this command may return fine without being able to > shut down WebHCat server Ambari performs another round of check to see if the > WebHCat process is still alive and kill it . > > In case `/usr/hdp/current/hive-webhcat/sbin/webhcat_server.sh stop` raised an > exception the stopping sequence exited early before the logic that would > force-kill WebHCat got executed. > > Modified the stop procedure such as always verifies at the end if the process > is still alive and kills it. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/webhcat_service.py > 487f078 > ambari-server/src/test/python/stacks/2.0.6/HIVE/test_webhcat_server.py > 86f185d > > Diff: https://reviews.apache.org/r/45833/diff/ > > > Testing > ------- > > Manual testing. > > Ambari server python unit tests: > ---------------------------------------------------------------------- > Total run:944 > Total errors:0 > Total failures:0 > > > Thanks, > > Sebastian Toader > >