----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62336/#review185496 -----------------------------------------------------------
Patch looks great! Reviews applied: [62252, 62230, 62343, 62344, 62336] 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 Sept. 15, 2017, 2 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62336/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2017, 2 a.m.) > > > Review request for mesos, Benjamin Mahler, Jie Yu, and Vinod Kone. > > > Bugs: MESOS-7939 > https://issues.apache.org/jira/browse/MESOS-7939 > > > Repository: mesos > > > Description > ------- > > This patch calls `Slave::checkDiskUsage` after `Slave::recover` > so completed executors are scheduled for GC to clean up disk space for > status updates of terminated executors in `Slave::_recover`. > > > Diffs > ----- > > src/slave/slave.hpp 7d07868451e93d34ba694d40216c1e4036fd4094 > src/slave/slave.cpp 6d1516a5d5b5db684f79385e60d892ff75fd00fd > > > Diff: https://reviews.apache.org/r/62336/diff/2/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Chun-Hung Hsiao > >