----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5072/#review7722 -----------------------------------------------------------
src/common/utils.hpp <https://reviews.apache.org/r/5072/#comment17010> Why not return an error? src/common/utils.hpp <https://reviews.apache.org/r/5072/#comment17007> Newline. src/slave/slave.hpp <https://reviews.apache.org/r/5072/#comment17008> const & src/slave/slave.cpp <https://reviews.apache.org/r/5072/#comment17012> IIUC, at this point, no executors should be live. src/slave/slave.cpp <https://reviews.apache.org/r/5072/#comment17011> ? src/slave/slave.cpp <https://reviews.apache.org/r/5072/#comment17009> Clock::now. - Benjamin On 2012-05-09 02:10:27, Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5072/ > ----------------------------------------------------------- > > (Updated 2012-05-09 02:10:27) > > > Review request for mesos, Benjamin Hindman, John Sirois, and Brian Wickman. > > > Summary > ------- > > This is the first cut for GC inside the slave. > > There are 2 kinds of gc going on > > --> Executor work dirs -- These get deleted whenever (after a timeout) an > executor exits/shutdown > --> Old slave dirs -- These get deleted when the slave gets registered for > the first time on a startup > > > Diffs > ----- > > src/slave/slave.hpp 08a29d8 > src/slave/slave.cpp 09a8396 > src/common/utils.hpp 1d81e21 > src/slave/constants.hpp f0c8679 > > Diff: https://reviews.apache.org/r/5072/diff > > > Testing > ------- > > make check scucceeds. > > Yet to write GC specific tests. > > > Thanks, > > Vinod > >
