> On Oct. 1, 2015, 6:59 p.m., Ben Mahler wrote: > > Could you please split the changes here? Specifically it looks like pulling > > up the structs and the chagnes to control flow are independent.
Done. Created https://reviews.apache.org/r/38941 - Jojy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38579/#review101267 ----------------------------------------------------------- On Oct. 1, 2015, 6:39 p.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38579/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2015, 6:39 p.m.) > > > Review request for mesos, Ben Mahler and Timothy Chen. > > > Repository: mesos > > > Description > ------- > > - Moved ManifestResponse struct from RegistryClient to namespace. > - Cleanup > > > Diffs > ----- > > src/slave/containerizer/provisioner/docker/registry_client.hpp > 9d5d15455192e2d75fe5cd7fa8755fb8cc67e185 > src/slave/containerizer/provisioner/docker/registry_client.cpp > c2040b48ea43fdb29766994c244273d3fa9ee3cd > src/tests/containerizer/provisioner_docker_tests.cpp > d895eb9d0723e52cff8b21ef2deeaef1911d019c > > Diff: https://reviews.apache.org/r/38579/diff/ > > > Testing > ------- > > make check. > > > Thanks, > > Jojy Varghese > >