----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10604/ -----------------------------------------------------------
(Updated April 18, 2013, 11:46 p.m.) Review request for mesos, Benjamin Hindman and Ben Mahler. Changes ------- benm's 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 (updated) ----- 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
