----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61094/#review181303 -----------------------------------------------------------
Patch looks great! Reviews applied: [61094] Passed command: support\windows-build.bat - Mesos Reviewbot Windows On July 24, 2017, 11:28 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61094/ > ----------------------------------------------------------- > > (Updated July 24, 2017, 11:28 p.m.) > > > Review request for mesos and Joseph Wu. > > > Bugs: MESOS-7252 > https://issues.apache.org/jira/browse/MESOS-7252 > > > Repository: mesos > > > Description > ------- > > With multi-role support, resources have an explicit notion of > an allocation role. The allocation role is used in the `contains` check > for example. The task/executor resources that the long-lived-framework > uses therefore need to be updated such that the check works correctly. > > > Diffs > ----- > > src/examples/long_lived_framework.cpp > bfc71c5e98b6403fc9548d857d89863871655991 > > > Diff: https://reviews.apache.org/r/61094/diff/2/ > > > Testing > ------- > > Manually observed that the `long-lived-framework` successfully launches tasks. > > > Thanks, > > Michael Park > >
