----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12885/#review24273 -----------------------------------------------------------
src/master/flags.hpp <https://reviews.apache.org/r/12885/#comment48102> s/" /"/ src/master/flags.hpp <https://reviews.apache.org/r/12885/#comment48103> s/" /"/ s/)/)./ src/master/master.cpp <https://reviews.apache.org/r/12885/#comment48099> Instead of overriding implicitly, can you change this to use EXIT(1)? EXIT(1) << "..."; It would be better to be explicit about the fact that their flag value is invalid. src/master/master.cpp <https://reviews.apache.org/r/12885/#comment48101> In Master::deactivateSlave, there's a WARNING for this, any reason to add this additional WARNING? - Ben Mahler On July 23, 2013, 11:42 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12885/ > ----------------------------------------------------------- > > (Updated July 23, 2013, 11:42 p.m.) > > > Review request for mesos. > > > Repository: mesos-git > > > Description > ------- > > Slave ping timeout may be increased by flags. > > Review: https://reviews.apache.org/r/12885 > > > Diffs > ----- > > src/master/constants.hpp f3a95f1006472ff2b90cabb8cf74182830740954 > src/master/constants.cpp 7bc32eb3bdae717ded63436e47031934384eec23 > src/master/flags.hpp 6dbbb79d447991a4929fe27628961c0874adadf8 > src/master/master.cpp e4507acd97ebaf2ce693ec7343e9a4a563f6ff80 > src/tests/fault_tolerance_tests.cpp > c8d88d5f60cf49ef5c1ffa429d308c30c2e5005c > > Diff: https://reviews.apache.org/r/12885/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Brenden Matthews > >
