----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11798/#review21715 -----------------------------------------------------------
src/slave/slave.cpp <https://reviews.apache.org/r/11798/#comment44880> Can you add a TODO here? Once we launch slave recovery, we don't want to allow these updates for tasks not found in the executor to be forwarded to the master / frameworks.. right? src/slave/slave.cpp <https://reviews.apache.org/r/11798/#comment44881> Again a TODO here? Just want to make sure the cleanup is made easier once we disallow the duplicate terminal updates. - Ben Mahler On June 11, 2013, 5:48 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11798/ > ----------------------------------------------------------- > > (Updated June 11, 2013, 5:48 a.m.) > > > Review request for mesos. > > > Description > ------- > > See summary. > > > This addresses bug MESOS-502. > https://issues.apache.org/jira/browse/MESOS-502 > > > Diffs > ----- > > src/slave/slave.cpp b5b7e0e44b3aced60ba2e800e05a7f9c0c8f6e5e > src/tests/status_update_manager_tests.cpp > 61ccfcc8165337d02af5641884beee45a2aaa065 > > Diff: https://reviews.apache.org/r/11798/diff/ > > > Testing > ------- > > make check > > sudo GLOG_v=1 ./bin/mesos-tests.sh > --gtest_filter="*DuplicateTerminalUpdates*" --verbose --gtest_repeat=1000 > --gtest_break_on_failure > > > Thanks, > > Vinod Kone > >
