----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43701/#review119623 -----------------------------------------------------------
src/slave/flags.cpp (line 693) <https://reviews.apache.org/r/43701/#comment180973> One space before `\n`, otherwise the word would be mixed with the first word of the next line. src/slave/flags.cpp (line 694) <https://reviews.apache.org/r/43701/#comment180974> ditto src/slave/slave.cpp (line 3709) <https://reviews.apache.org/r/43701/#comment180972> Instead of repeating the `if (flags.http_command_executor)...` logic multiple times, I would prefer use a temp variable to store either `mesos-executor` or `mesos-http-executor`. - Shuai Lin On Feb. 18, 2016, 1:09 a.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43701/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2016, 1:09 a.m.) > > > Review request for mesos, Anand Mazumdar and Vinod Kone. > > > Bugs: MESOS-3558 > https://issues.apache.org/jira/browse/MESOS-3558 > > > Repository: mesos > > > Description > ------- > > Added a command executor based on the new V1 API. > > > Diffs > ----- > > docs/configuration.md 3d8236822af688a88a9f9f357c67c03d7d60fdd9 > include/mesos/v1/mesos.proto d909e60ddfd8e3ba2075f82c372edde04cd99d54 > src/Makefile.am 54ebe91643a17017c00cdbd5dfc8ce1a021579d5 > src/launcher/http_executor.cpp PRE-CREATION > src/slave/flags.hpp bd52b4f9816b271a9d2728ae6915f8e24b74d716 > src/slave/flags.cpp d4b4e5201bb3a7e8edd0ab2481328fc91cd8f974 > src/slave/slave.cpp f0be0d5bf4b853952caf06e2e262c2903d79ead3 > > Diff: https://reviews.apache.org/r/43701/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Qian Zhang > >
