----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58422/ -----------------------------------------------------------
(Updated May 4, 2017, 10:52 a.m.) Review request for atlas and David Radley. Changes ------- Have addressed David's review comments - added code comment and improved logged messages Summary (updated) ----------------- Addressed Davids review comments Repository: atlas Description ------- On Windows there is no SIGKILL in the python signal library, so attempting to import SIGKILL causes a fatal error. This change introduces a platform switch that avoids trying to import SIGKILL and if the 30sec timeout expires it avoids attempting to use SIGKILL, instead using SIGTERM as the os.kill() function on Windows will terminate the process and should be as severe as a SIGKILL (kill -9) on a *nix system. Diffs (updated) ----- distro/src/bin/atlas_stop.py a25d25aee599e7cc9ca3caaff8ff7f11b7e0c789 Diff: https://reviews.apache.org/r/58422/diff/2/ Changes: https://reviews.apache.org/r/58422/diff/1-2/ Testing (updated) ------- Manually tested Thanks, Graham Wallis