---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review189670
---
Ship it!
Ship It!
- Vinod Kone
On Oct. 12, 2017, 12:57
> On Oct. 12, 2017, 4:10 p.m., Benjamin Bannier wrote:
> > Does it makes sense to add a test for this?
Yeah, I added a test here: https://reviews.apache.org/r/62837/
- Qian
---
This is an automatically generated e-mail. To reply,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review187752
---
Does it makes sense to add a test for this?
- Benjamin Bannier
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review187716
---
src/launcher/default_executor.cpp
Line 894 (original), 903
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review187384
---
Patch looks great!
Reviews applied: [62836]
Passed command:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review187376
---
FAIL: The system cannot find the file specified.
Reviews
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/#review187377
---
FAIL: The system cannot find the file specified.
Reviews
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62836/
---
Review request for mesos and Vinod Kone.
Bugs: MESOS-8051