Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-13 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/#review197478 --- Patch looks great! Reviews applied: [65445, 65504, 65446, 65449,

Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-13 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/#review197476 --- PASS: Mesos patch 65593 was successfully built and tested.

Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-13 Thread Meng Zhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/ --- (Updated Feb. 13, 2018, 7:33 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-13 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/#review197470 --- Fix it, then Ship it! src/tests/slave_tests.cpp Lines 4841

Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-13 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/#review197388 --- PASS: Mesos patch 65593 was successfully built and tested.

Re: Review Request 65593: Added tests to check executor that failed to launch is removed.

2018-02-12 Thread Meng Zhu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65593/ --- (Updated Feb. 12, 2018, 10:26 p.m.) Review request for mesos and Vinod Kone.