> On June 27, 2016, 4:35 p.m., Zhitao Li wrote: > > include/mesos/v1/master/master.proto, line 296 > > <https://reviews.apache.org/r/49136/diff/5/?file=1430594#file1430594line296> > > > > Should this be required? I guess that's the case for any custom > > executor type, but I'm not sure about CommandExecutor.
I'll fix this while committing. While command executors do not have ExecutorInfo here, the ones that do have executors should have ExecutorInfo. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49136/#review139592 ----------------------------------------------------------- On June 27, 2016, 7:47 a.m., zhou xing wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49136/ > ----------------------------------------------------------- > > (Updated June 27, 2016, 7:47 a.m.) > > > Review request for mesos, Anand Mazumdar, haosdent huang, and Vinod Kone. > > > Bugs: mesos-5492 > https://issues.apache.org/jira/browse/mesos-5492 > > > Repository: mesos > > > Description > ------- > > Add 'Framework' protobuf message to master/master.proto & > v1/master/master.proto. > > > Diffs > ----- > > include/mesos/master/master.proto 639fbd110df4aca1cf700cb9e455eecc110a7f66 > include/mesos/v1/master/master.proto > 11dfab318eb073908a9e302afa33b274fec63a16 > > Diff: https://reviews.apache.org/r/49136/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > zhou xing > >
