----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56778/#review165910 -----------------------------------------------------------
Patch looks great! Reviews applied: [56778] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Feb. 17, 2017, 5:24 a.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56778/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2017, 5:24 a.m.) > > > Review request for mesos and Benjamin Mahler. > > > Bugs: MESOS-7137 > https://issues.apache.org/jira/browse/MESOS-7137 > > > Repository: mesos > > > Description > ------- > > We were not unallocating the resources before checking if the > executor resources were contained in the checkpointed resources > on the agent. > > > Diffs > ----- > > src/slave/slave.cpp ebba8e16bc9ec45781183e78cb5a3c351a5f65f5 > src/tests/default_executor_tests.cpp > ffb69e9d6745c267704195056500edd4d8a4ca3f > > Diff: https://reviews.apache.org/r/56778/diff/ > > > Testing > ------- > > make check + added test that fails without the change. > > > Thanks, > > Anand Mazumdar > >
