----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49419/#review140186 -----------------------------------------------------------
Fix it, then Ship it! LGTM, modulo earlier comments from Zhitao. include/mesos/master/master.proto (lines 343 - 346) <https://reviews.apache.org/r/49419/#comment205503> Can we include documentation on what each of these mean for posterity? Ditto for the v1 protobuf. src/master/http.cpp (lines 3197 - 3198) <https://reviews.apache.org/r/49419/#comment205507> whoops, this is leaking memory. - Anand Mazumdar On June 30, 2016, 1:35 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49419/ > ----------------------------------------------------------- > > (Updated June 30, 2016, 1:35 a.m.) > > > Review request for mesos, Anand Mazumdar, Jay Guo, and Zhitao Li. > > > Bugs: MESOS-5493 > https://issues.apache.org/jira/browse/MESOS-5493 > > > Repository: mesos > > > Description > ------- > > The response now distinguishes between active tasks, completed tasks, > pending tasks and orphan tasks to make it easy for clients. > Consequently got rid of offset, limit and offset in the Call because > they don't make sense when we have multiple fields in the response. > > > Diffs > ----- > > include/mesos/master/master.proto 2e5d6eeb3a960e4f41c382d65321f18bb05ed6be > include/mesos/v1/master/master.proto > 93157d57dcc53b54fed2ebbc4772c689ddba2119 > src/master/http.cpp e5acdb8e0bbcd7a2b7e8a8bc7f4bbeaae2c4fea1 > src/master/validation.cpp 6939d0e6ac3dbddc10f0315bd0a696a1f71634e2 > src/tests/api_tests.cpp b3127b716480223a6f23b7908bf6bc1808120f80 > > Diff: https://reviews.apache.org/r/49419/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
