Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-25 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40681/#review108021 --- Ship it! Will fix the issue Bernd noted and commit. - Till Toensh

Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-25 Thread Till Toenshoff
> On Nov. 25, 2015, 11:57 a.m., Bernd Mathiske wrote: > > src/local/local.cpp, line 373 > > > > > > !isSome() -> isNone() Will fix while committing. - Till ---

Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-25 Thread Jojy Varghese
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40681/ --- (Updated Nov. 25, 2015, 4:01 p.m.) Review request for mesos, Jie Yu and Timothy

Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40681/#review107973 --- Ship it! src/local/local.cpp (line 373)

Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-25 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40681/#review107960 --- Patch looks great! Reviews applied: [40681] Passed command: expor

Re: Review Request 40681: Changed mesos-local to use posix launcher as default.

2015-11-24 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40681/#review107890 --- Ship it! Ship It! - haosdent huang On Nov. 25, 2015, 12:38 a.m.