You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Jojy Varghese <jo...@mesosphere.io> on 2015/11/01 16:37:53 UTC

Review Request 39840: RegistryClient refactor: pulled up streaming read

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39840/
-----------------------------------------------------------

Review request for mesos and Ben Mahler.


Repository: mesos


Description
-------

Made reading the stream from the PIPE common.


Diffs
-----

  src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e4d2c22cf6627c1c76ebafeeb84b2bbf6b8c238c 

Diff: https://reviews.apache.org/r/39840/diff/


Testing
-------

make check.


Thanks,

Jojy Varghese


Re: Review Request 39840: RegistryClient refactor: pulled up streaming read

Posted by Jojy Varghese <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39840/
-----------------------------------------------------------

(Updated Nov. 4, 2015, 11:24 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

rebased.


Repository: mesos


Description
-------

Made reading the stream from the PIPE common.


Diffs (updated)
-----

  src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e4d2c22cf6627c1c76ebafeeb84b2bbf6b8c238c 

Diff: https://reviews.apache.org/r/39840/diff/


Testing
-------

make check.


Thanks,

Jojy Varghese


Re: Review Request 39840: RegistryClient refactor: pulled up streaming read

Posted by Jojy Varghese <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39840/
-----------------------------------------------------------

(Updated Nov. 1, 2015, 3:52 p.m.)


Review request for mesos and Ben Mahler.


Changes
-------

removed overloaded methd.


Repository: mesos


Description
-------

Made reading the stream from the PIPE common.


Diffs (updated)
-----

  src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e4d2c22cf6627c1c76ebafeeb84b2bbf6b8c238c 

Diff: https://reviews.apache.org/r/39840/diff/


Testing
-------

make check.


Thanks,

Jojy Varghese