----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54281/#review158470 -----------------------------------------------------------
Fix it, then Ship it! src/tests/api_tests.cpp (lines 3769 - 3780) <https://reviews.apache.org/r/54281/#comment229238> Small nit, but could you put the strings above the `Call` declaration to keep the call declaration and its field assignments all grouped together? - Kevin Klues On Dec. 7, 2016, 7:19 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54281/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2016, 7:19 p.m.) > > > Review request for mesos, Anand Mazumdar, Jie Yu, and Kevin Klues. > > > Repository: mesos > > > Description > ------- > > This test verifies that launch results in output being streamed. > > > Diffs > ----- > > src/tests/api_tests.cpp 6cd1f83a47731ee8c4fb1f022bf585b271b3b966 > > Diff: https://reviews.apache.org/r/54281/diff/ > > > Testing > ------- > > make -j20 check GTEST_FILTER="*LaunchNestedContainerSession*" GLOG_v=1 > MESOS_VERBOSE=1 > > > Thanks, > > Vinod Kone > >
