----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63232/#review189029 -----------------------------------------------------------
Patch looks great! Reviews applied: [63232] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Oct. 23, 2017, 11:56 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63232/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2017, 11:56 p.m.) > > > Review request for mesos, Benjamin Mahler and Greg Mann. > > > Bugs: MESOS-7851 > https://issues.apache.org/jira/browse/MESOS-7851 > > > Repository: mesos > > > Description > ------- > > The intention is for us to work with resources in the > `post-reservation-refinement` format > in the master memory, and store them in the master registry or agent > checkpoint in > the `pre` format in order to support downgrades. In this case, we correctly > store the `pre` > format resources in the master registry, but we don't convert them back when > we read it out > of the registry. This patch converts the resources back to `post` when we > read from the registry. > > > Diffs > ----- > > src/master/master.cpp 4b76648b4887724e166ff95e002ee26c8bfef006 > > > Diff: https://reviews.apache.org/r/63232/diff/1/ > > > Testing > ------- > > `make check` > > > Thanks, > > Michael Park > >
