mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben Mahler" <benjamin.mah...@gmail.com>
Subject Re: Review Request 37327: Added a recordio::Reader for wrapping an http::Pipe::Reader.
Date Tue, 11 Aug 2015 00:41:03 GMT

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

(Updated Aug. 11, 2015, 12:41 a.m.)


Review request for mesos, Anand Mazumdar, Benjamin Hindman, Jie Yu, and Vinod Kone.


Changes
-------

Added documentation and improved one of the tests. No need for further reviews.


Repository: mesos


Description
-------

This is helpful for the scheduler library and tests. Since we
currently do not have a generalized abstraction in libprocess
for "streams" of asynchronous data (e.g. process::Stream<T>),
we have to create a one-off wrapper here.


Diffs (updated)
-----

  src/Makefile.am 942003149b071a322933e2c085d9122903f69713 
  src/common/recordio.hpp PRE-CREATION 
  src/tests/common/recordio_tests.cpp PRE-CREATION 

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


Testing
-------

Added tests.


Thanks,

Ben Mahler


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message