----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review77536 -----------------------------------------------------------
include/mesos/mesos.proto <https://reviews.apache.org/r/32139/#comment125648> Not sure this is the right place to comment on the valid specifications of both role and reservation. There is a user guide added right? We should refer to that instead. include/mesos/mesos.proto <https://reviews.apache.org/r/32139/#comment125649> It also can be unreserved as well? - Timothy Chen On March 18, 2015, 7:43 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32139/ > ----------------------------------------------------------- > > (Updated March 18, 2015, 7:43 p.m.) > > > Review request for mesos, Alexander Rukletsov, Ben Mahler, and Jie Yu. > > > Bugs: MESOS-2475 > https://issues.apache.org/jira/browse/MESOS-2475 > > > Repository: mesos > > > Description > ------- > > The beginning of `Resource::ReservationInfo`. This patch only introduces the > `framework_id` field. We don't need the required `principal` field yet and > will be introduced together with the "unreserve" ACLs. An optional `id` may > be introduced in the future as well. > > > Diffs > ----- > > include/mesos/mesos.proto ec8efaec13f54a56d82411f6cdbdb8ad8b103748 > > Diff: https://reviews.apache.org/r/32139/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >