----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10604/#review19424 -----------------------------------------------------------
Ship it! Ship It! - Ben Mahler On April 19, 2013, 1:28 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10604/ > ----------------------------------------------------------- > > (Updated April 19, 2013, 1:28 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > Refactored runTask() and some other pieces of slave, to make this hopefully > clear. > > Also, sneaked in some bug fixes when executorStarted() is called. > > > Diffs > ----- > > src/slave/slave.hpp 54c66863db217077a050dc414caf0976447500be > src/slave/slave.cpp 00b2375505e362959ac34061e3066cf8ace96adf > src/tests/allocator_zookeeper_tests.cpp > 42faaa067bdfa0c7f33260eb5cb3b9e5956c3037 > > Diff: https://reviews.apache.org/r/10604/diff/ > > > Testing > ------- > > make check. > > NOTE: GarbageCollectorIntegrationTest.Unschedule test now correctly verifies > that executors/frameworks are properly unscheduled despite adding tasks to > 'pending'. > > > Thanks, > > Vinod Kone > >
