----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102579 -----------------------------------------------------------
Patch looks great! Reviews applied: [39013, 38443, 39184, 39155, 39156, 38579, 39014, 39068, 38941, 39015, 39016, 39017, 39053, 39112, 38580, 38747, 39250] All tests passed. - Mesos ReviewBot On Oct. 14, 2015, 12:45 a.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39250/ > ----------------------------------------------------------- > > (Updated Oct. 14, 2015, 12:45 a.m.) > > > Review request for mesos and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > Puller refactor: moved untar to a common place > > > Diffs > ----- > > src/slave/containerizer/provisioner/docker/local_puller.cpp > 74d0e1ead7d630e65a7e75cb6123139b9197efef > src/slave/containerizer/provisioner/docker/puller.hpp > 105b4e75439c2ad4c08e2fd364f288f1d39b9b59 > src/slave/containerizer/provisioner/docker/puller.cpp > cb05324689ffa26ce830b513e2d71b55517da3cb > src/slave/containerizer/provisioner/docker/remote_puller.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/39250/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Jojy Varghese > >