----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37773/#review97138 -----------------------------------------------------------
src/slave/containerizer/provisioners/docker/registry_client.hpp (line 111) <https://reviews.apache.org/r/37773/#comment152903> We are only fetching one blob each call right? so s/fetched blob/fetched blobs/g ? src/slave/containerizer/provisioners/docker/registry_client.cpp (line 456) <https://reviews.apache.org/r/37773/#comment152904> is maxSize being used at all? If not we should remove all references of it. - Timothy Chen On Aug. 30, 2015, 3:12 p.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37773/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2015, 3:12 p.m.) > > > Review request for mesos, Lily Chen and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > Added implementation for docker registry's Get Manifest and Get Blob APIs. > > > Diffs > ----- > > src/Makefile.am 7b620ff66856b3f0adac121b3297d55ed71a3d99 > src/slave/containerizer/provisioners/docker/registry_client.hpp > PRE-CREATION > src/slave/containerizer/provisioners/docker/registry_client.cpp > PRE-CREATION > src/tests/provisioners/docker_provisioner_tests.cpp PRE-CREATION > > Diff: https://reviews.apache.org/r/37773/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jojy Varghese > >
