----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66652/#review201421 -----------------------------------------------------------
Fix it, then Ship it! src/tests/containerizer/runtime_isolator_tests.cpp Lines 178-181 (original), 178-181 (patched) <https://reviews.apache.org/r/66652/#comment282654> I think we can merge these 4 lines of code into 3 lines. - Qian Zhang On April 17, 2018, 1:32 p.m., Gilbert Song wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66652/ > ----------------------------------------------------------- > > (Updated April 17, 2018, 1:32 p.m.) > > > Review request for mesos, Jie Yu and Qian Zhang. > > > Bugs: MESOS-8794 > https://issues.apache.org/jira/browse/MESOS-8794 > > > Repository: mesos > > > Description > ------- > > Renamed local_puller to image_tar_puller. > > > Diffs > ----- > > src/CMakeLists.txt 6fc45dba8ee70b591be03ac483655c1844a0a6b9 > src/Makefile.am 9f4b6d369a23af337e2384e52e3e41f4017df38a > src/slave/containerizer/mesos/provisioner/docker/local_puller.hpp > 4d2e4973a0d6c99dd3447a158003b4b09e2ba477 > src/slave/containerizer/mesos/provisioner/docker/local_puller.cpp > 5ce49ac396b03e8b6d87601ecaa0691d88de21e3 > src/slave/containerizer/mesos/provisioner/docker/puller.cpp > d7d8987d493a37d20f32ddd254dc0c3b15159951 > src/tests/containerizer/provisioner_docker_tests.cpp > c664ff807583d587d94b0ab797330d5d3daf7657 > src/tests/containerizer/runtime_isolator_tests.cpp > b703b827a9a00b1c335cd1773c4d3e048eb16d66 > > > Diff: https://reviews.apache.org/r/66652/diff/1/ > > > Testing > ------- > > > Thanks, > > Gilbert Song > >
