----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36470/#review91773 -----------------------------------------------------------
src/sched/sched.cpp (line 852) <https://reviews.apache.org/r/36470/#comment145419> Ditto newline comment. src/tests/exception_tests.cpp (lines 195 - 196) <https://reviews.apache.org/r/36470/#comment145421> Is mesos:: needed? src/tests/master_allocator_tests.cpp (line 445) <https://reviews.apache.org/r/36470/#comment145422> Is mesos:: needed? src/tests/master_tests.cpp (lines 246 - 247) <https://reviews.apache.org/r/36470/#comment145423> Is mesos:: needed? src/tests/master_tests.cpp (line 249) <https://reviews.apache.org/r/36470/#comment145424> We used to write silly comments.. ;) src/tests/slave_recovery_tests.cpp (lines 2154 - 2155) <https://reviews.apache.org/r/36470/#comment145426> Is mesos:: needed? - Ben Mahler On July 14, 2015, 12:30 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36470/ > ----------------------------------------------------------- > > (Updated July 14, 2015, 12:30 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2913 > https://issues.apache.org/jira/browse/MESOS-2913 > > > Repository: mesos > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/sched/sched.cpp a748686dfc6bff39d81fd7adbd5cce88ddaaa73d > src/tests/exception_tests.cpp 9af16748ae67671bd327a1ea7c946a7a38c5f7ff > src/tests/master_allocator_tests.cpp > 534b2486a6a02ef32b5a7235d3ad30e82a78d6a5 > src/tests/master_tests.cpp 57721b788d0c70f4c6f5cc44d87465f52a70b6c2 > src/tests/rate_limiting_tests.cpp 49d907b0be45ccfed8af5c8fda89ad560e012c1e > src/tests/slave_recovery_tests.cpp 2f882cf7b4235583b0ec8397cfcbbc02930fbc88 > > Diff: https://reviews.apache.org/r/36470/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
