Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-09-18 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51443/ --- (Updated Sept. 18, 2016, 6:55 p.m.) Review request for mesos and Jie Yu. Bugs

Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-08-26 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51443/ --- (Updated Aug. 26, 2016, 9:42 p.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-08-26 Thread Kevin Klues
> On Aug. 26, 2016, 9:04 p.m., Jie Yu wrote: > > src/slave/containerizer/mesos/linux_launcher.cpp, lines 497-508 > > > > > > What if it is partially written? Should we return None() in that case > > as well (maybe p

Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-08-26 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51443/ --- (Updated Aug. 26, 2016, 9:39 p.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-08-26 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51443/#review147021 --- Fix it, then Ship it! src/slave/containerizer/mesos/launcher.c

Re: Review Request 51443: Updated 'launcher' to properly reap checkpointed exit status on linux.

2016-08-26 Thread Kevin Klues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51443/ --- (Updated Aug. 26, 2016, 8:40 p.m.) Review request for mesos and Jie Yu. Chang