----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57058/#review166799 -----------------------------------------------------------
Fix it, then Ship it! src/tests/default_executor_tests.cpp (line 620) <https://reviews.apache.org/r/57058/#comment238872> Nit: Would fit in one line? src/tests/default_executor_tests.cpp (line 1038) <https://reviews.apache.org/r/57058/#comment238873> Nit: Would fit in one line? src/tests/default_executor_tests.cpp (line 1189) <https://reviews.apache.org/r/57058/#comment238874> Nit: Would fit in one line? - Anand Mazumdar On Feb. 25, 2017, 2:01 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57058/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2017, 2:01 a.m.) > > > Review request for mesos, Anand Mazumdar and Gilbert Song. > > > Repository: mesos > > > Description > ------- > > Now all the tests in this file use v1 protos. > > > Diffs > ----- > > src/tests/default_executor_tests.cpp > eaf639467aaaa35b28b21bfb7e16aca5924a5a82 > > Diff: https://reviews.apache.org/r/57058/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
