> On Feb. 21, 2017, 8:42 p.m., Vinod Kone wrote: > > include/mesos/mesos.proto, line 655 > > <https://reviews.apache.org/r/56831/diff/1/?file=1640430#file1640430line655> > > > > Can you also update the docs that say it only accepts a single > > executor? Also, worth calling out in 1.3 CHANGELOG (WIP).
Updated the nested task groups doc. Would update the CHANGELOG in a separate patch. - Anand ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56831/#review166219 ----------------------------------------------------------- On Feb. 22, 2017, 6:16 p.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56831/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2017, 6:16 p.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Modified a comment now that the default executor can launch > multiple task groups. Also, updated the nested task groups > docs to reflect this. > > > Diffs > ----- > > docs/nested-container-and-task-group.md > debf9f82622557aff915dd04ef06a50793f496b6 > include/mesos/mesos.proto 030e79c003f6560e9c0627db12fb1baba411151d > include/mesos/v1/mesos.proto 7f6f858a7d9387d202510730d400e490298e6574 > > Diff: https://reviews.apache.org/r/56831/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Anand Mazumdar > >
