----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59861/#review178074 -----------------------------------------------------------
include/mesos/mesos.proto Lines 356-357 (patched) <https://reviews.apache.org/r/59861/#comment251875> > // Without this capability, a framework simply is not offered resources > // with reservation refinement. "Without this capability, a framework simply is not offered resources that have reservation refinement" include/mesos/mesos.proto Lines 835 (patched) <https://reviews.apache.org/r/59861/#comment251876> s/with reservation refinements/that have reservation refinement/ include/mesos/mesos.proto Line 925 (original), 947-950 (patched) <https://reviews.apache.org/r/59861/#comment251878> Newline between "sections" include/mesos/mesos.proto Lines 969-970 (patched) <https://reviews.apache.org/r/59861/#comment251879> `s/Two Resource Formats/Resource Format:/` include/mesos/mesos.proto Lines 998 (patched) <https://reviews.apache.org/r/59861/#comment251880> Connect. include/mesos/v1/mesos.proto Lines 829 (patched) <https://reviews.apache.org/r/59861/#comment251877> s/with reservation refinements/that have reservation refinement/ include/mesos/v1/mesos.proto Line 920 (original), 942-943 (patched) <https://reviews.apache.org/r/59861/#comment251881> Copy over the changes from the V0 API. - Michael Park On June 15, 2017, 12:37 a.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59861/ > ----------------------------------------------------------- > > (Updated June 15, 2017, 12:37 a.m.) > > > Review request for mesos, Benjamin Mahler and Neil Conway. > > > Bugs: MESOS-7575 > https://issues.apache.org/jira/browse/MESOS-7575 > > > Repository: mesos > > > Description > ------- > > With reservation refinement, we now represent the reservation state in > `Resource.reservations` rather than with `Resource.role` and > `Resource.reservation`. We also keep track of the type of reservation, > and the role the reservation is for in the `Resource.ReservationInfo`. > This patch introduces these changes and explains the different formats > in detail. > > > Diffs > ----- > > include/mesos/mesos.proto e9a8b1513150b394f4986a1f34483cfca32b35e3 > include/mesos/v1/mesos.proto 5e8c15c07a02999e3376d2104c1c642f00511969 > src/common/protobuf_utils.hpp 7b1a1289c16edde44143432b645ecc2222e68576 > src/common/protobuf_utils.cpp 46331b379c732edc597b46bb10665c95c9640e96 > src/slave/constants.cpp d3668e4dca515f42d88d01b0852a543bfdb360f6 > > > Diff: https://reviews.apache.org/r/59861/diff/9/ > > > Testing > ------- > > Updated tests + `make check` > > > Thanks, > > Michael Park > >
