----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43701/#review119803 -----------------------------------------------------------
src/launcher/http_executor.cpp (line 776) <https://reviews.apache.org/r/43701/#comment181154> Unlike unacked status updates, there could be at most one unacked task info, so here we don't need a map of tasks, maybe an `Option<v1::TaskInfo>` is enough. I know that the old `mesos::internal::ExecutorProcess` uses a map to store the unacked tasks, but that's because it's part of the generic `ExecutorDriver`, which is designed to support launching multiple tasks with the same executor, while the mesos cmd executor could only launch one task. - 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 > >
