----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49464/#review140256 -----------------------------------------------------------
Fix it, then Ship it! Other than sequence number, looks good. include/mesos/master/master.proto (line 364) <https://reviews.apache.org/r/49464/#comment205608> same, sequence number should be 1 include/mesos/v1/master/master.proto (line 365) <https://reviews.apache.org/r/49464/#comment205607> The sequence number should be 1 - Zhitao Li On June 30, 2016, 7:04 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49464/ > ----------------------------------------------------------- > > (Updated June 30, 2016, 7:04 p.m.) > > > Review request for mesos, Abhishek Dasgupta, Anand Mazumdar, zhou xing, Jay > Guo, Shuai Lin, Vinod Kone, and Zhitao Li. > > > Bugs: MESOS-5750 > https://issues.apache.org/jira/browse/MESOS-5750 > > > Repository: mesos > > > Description > ------- > > Updated operator APIs proto files for `GET_EXECUTORS` call. > > > Diffs > ----- > > include/mesos/master/master.proto 2e5d6eeb3a960e4f41c382d65321f18bb05ed6be > include/mesos/v1/master/master.proto > 93157d57dcc53b54fed2ebbc4772c689ddba2119 > > Diff: https://reviews.apache.org/r/49464/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >