-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45833/#review127570
-----------------------------------------------------------


Ship it!




Ship It!

- Dmytro Sen


On Апрель 6, 2016, 8:21 п.п., Sebastian Toader wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45833/
> -----------------------------------------------------------
> 
> (Updated Апрель 6, 2016, 8:21 п.п.)
> 
> 
> 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
>  10e0990 
>   ambari-server/src/test/python/stacks/2.0.6/HIVE/test_webhcat_server.py 
> 6fe49dc 
> 
> 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
> 
>

Reply via email to