Re: Review Request 69681: Fixed the FD leak if containerizer::_launch() failed or discarded.

2019-01-09 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69681/#review211792 --- FAIL: Some of the unit tests failed. Please check the relevant

Re: Review Request 69681: Fixed the FD leak if containerizer::_launch() failed or discarded.

2019-01-07 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69681/#review211726 --- src/slave/containerizer/mesos/containerizer.cpp Lines 1375

Re: Review Request 69681: Fixed the FD leak if containerizer::_launch() failed or discarded.

2019-01-07 Thread Qian Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69681/#review211724 --- include/mesos/slave/containerizer.hpp Lines 85 (patched)

Review Request 69681: Fixed the FD leak if containerizer::_launch() failed or discarded.

2019-01-07 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69681/ --- Review request for mesos, Andrei Budnik, Jie Yu, Qian Zhang, and Vinod Kone.