----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51607/#review154569 -----------------------------------------------------------
Ship it! src/health-check/health_checker.cpp (line 77) <https://reviews.apache.org/r/51607/#comment224186> Some abstraction doing this automagically would be cool to have, no? e.g. `Path::runnable(string)` adding ".exe" on Windows. src/health-check/health_checker.cpp (line 588) <https://reviews.apache.org/r/51607/#comment224191> Not a big fan of such messages for our users. However I feel this surpasses the scope of this review and should be discussed in a more general way (if at all). - Till Toenshoff On Oct. 31, 2016, 6:56 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51607/ > ----------------------------------------------------------- > > (Updated Oct. 31, 2016, 6:56 p.m.) > > > Review request for mesos, Avinash sridharan, Benjamin Mahler, Gastón Kleiman, > and haosdent huang. > > > Bugs: MESOS-6119 > https://issues.apache.org/jira/browse/MESOS-6119 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/docker/executor.cpp ab3f0473fdc9105d1c425f0dbe7b81c566d541e8 > src/health-check/health_checker.hpp > 392b4d5bd1e5831994b9366c1eb5a2911e19860f > src/health-check/health_checker.cpp > 96ae1a733ff3d211b84d0893b4603873af1c89f0 > src/launcher/default_executor.cpp f5ff02a63d0b12833dd383378f99ee337a3bcf30 > src/launcher/executor.cpp 0544121e679db503fe4eaf23a24e315eb188a520 > > Diff: https://reviews.apache.org/r/51607/diff/ > > > Testing > ------- > > make check on Mac OS 10.11.6 > > > Thanks, > > Alexander Rukletsov > >
