> On Oct. 15, 2015, 10:51 p.m., Timothy Chen wrote: > > Ship It! > > Timothy Chen wrote: > This doesn't seem to depend on anything from 38747. I'll be merging this > without the previous one. > > Timothy Chen wrote: > Btw please rebase this patch.
The dependency is because RemotePuller has not yet landed. - Jojy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39250/#review102827 ----------------------------------------------------------- On Oct. 15, 2015, 10:54 p.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39250/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2015, 10:54 p.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 > >
