----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59861/#review178073 -----------------------------------------------------------
Ship it! Modulo comments. - Benjamin Mahler On June 15, 2017, 7: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, 7: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 > >
