----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53573/#review156568 -----------------------------------------------------------
Fix it, then Ship it! I'll fix the last nit and will commit for you shortly. src/tests/kill_policy_test_helper.hpp (line 36) <https://reviews.apache.org/r/53573/#comment226755> `<cstdint>` should be included in this file. - Alexander Rukletsov On Nov. 21, 2016, 12:59 p.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53573/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2016, 12:59 p.m.) > > > Review request for mesos and Alexander Rukletsov. > > > Repository: mesos > > > Description > ------- > > This patch adds a test helper to facilitate the testing of `KillPolicy`. > > The helper blocks until it receives a SIGTERM, then sleeps for a > configurable amount of time before finally returning EXIT_SUCCESS. > > > Diffs > ----- > > src/Makefile.am 5a47c93388234a68c3c486a021ccdbe3213c5bac > src/tests/CMakeLists.txt cf583ea3f10482b510459fb11a7ecaf76e498391 > src/tests/kill_policy_test_helper.hpp PRE-CREATION > src/tests/kill_policy_test_helper.cpp PRE-CREATION > src/tests/test_helper_main.cpp 42a8ec675500cb7b155823a4ea64d70a3481eaf4 > > Diff: https://reviews.apache.org/r/53573/diff/ > > > Testing > ------- > > `make check` (macOS and Debian) > > > Thanks, > > Gastón Kleiman > >
