----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38899/#review101439 -----------------------------------------------------------
src/slave/http.cpp (line 316) <https://reviews.apache.org/r/38899/#comment158829> I think that the blank line is not needed. - Guangya Liu On 九月 30, 2015, 5:02 p.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38899/ > ----------------------------------------------------------- > > (Updated 九月 30, 2015, 5:02 p.m.) > > > Review request for mesos, Isabel Jimenez and Vinod Kone. > > > Bugs: MESOS-2296 > https://issues.apache.org/jira/browse/MESOS-2296 > > > Repository: mesos > > > Description > ------- > > Trivial change to call the `executorMessage(...)` function that already > existed in `src/slave/slave.cpp` to handle ExecutorToFrameworkMessage. > > > Diffs > ----- > > src/slave/http.cpp f9cf7bbe81b7fe9637de9a8d66329c16a7e1a89b > > Diff: https://reviews.apache.org/r/38899/diff/ > > > Testing > ------- > > make check. Would add tests later after the MESOS-3480 review chain gets > committed. > > > Thanks, > > Anand Mazumdar > >