----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58422/#review178961 -----------------------------------------------------------
distro/src/bin/atlas_stop.py Lines 82 (patched) <https://reviews.apache.org/r/58422/#comment253355> formatting error introduced in the fix - David Radley On May 4, 2017, 10:52 a.m., Graham Wallis wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > distro/src/bin/atlas_stop.py a25d25aee599e7cc9ca3caaff8ff7f11b7e0c789 > > > Diff: https://reviews.apache.org/r/58422/diff/2/ > > > Testing > ------- > > Manually tested > > > Thanks, > > Graham Wallis > >
