----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10596/#review19360 -----------------------------------------------------------
src/slave/status_update_manager.hpp <https://reviews.apache.org/r/10596/#comment40061> As discussed, make this return Try<bool>: Whether the ack was processed. That way we can log warnings for ignored updates rather than failures. src/tests/status_update_manager_tests.cpp <https://reviews.apache.org/r/10596/#comment40062> I'd kill this, but up to you. src/tests/status_update_manager_tests.cpp <https://reviews.apache.org/r/10596/#comment40063> indent src/tests/status_update_manager_tests.cpp <https://reviews.apache.org/r/10596/#comment40064> Adjust the comment after you update to reflect it's no longer an "error". - Ben Mahler On April 18, 2013, 12:08 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10596/ > ----------------------------------------------------------- > > (Updated April 18, 2013, 12:08 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/slave/slave.cpp 9ddfc521799173511c43ded4dbd7a6f4e2c3b82a > src/slave/status_update_manager.hpp > a5d4f9e8c829e5ef63b16f8ab60efec1ebc1f73c > src/slave/status_update_manager.cpp > f03885d4f7486286af3b61eaab957a1334280c78 > src/tests/status_update_manager_tests.cpp > e51d689a63342f4995dfc4b28ba8fe6505788646 > src/tests/utils.hpp b5c577ddc3d4296fe8e77815cad158e2ef3bbbda > > Diff: https://reviews.apache.org/r/10596/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
