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

(Updated Oct. 10, 2012, 1:07 a.m.)


Review request for mesos, Benjamin Hindman and Vinod Kone.


Changes
-------

-Read size is capped at 2x the system page size.
-The pailer documentation is updated to reflect the fact that the 'length' is 
removed.
-Retested on linux, and with the webui pailer.


Summary (updated)
-----------------

Changing FilesProcess to use async io::read, updated pailer to not use 
redundant 'length' field of response.


Description
-------

see above

also:
-removed redundant length field from json
-added missing test for buffered io::read


Diffs (updated)
-----

  src/files/files.cpp c40b7b74485dc717c4291f17a4533b52f2c51a7e 
  src/tests/files_tests.cpp 17d284b82de4d9e1dedf1c26f4cedcf8269c4072 
  src/webui/master/static/jquery.pailer.js 
edd23d9efa03086679af67a8bd13a273d409798b 
  third_party/libprocess/include/process/io.hpp 
722b15f11db4d3b6c941c8b4a6c5d395b11b035f 
  third_party/libprocess/src/process.cpp 
fde7154f9a4432acaff09849c0fa69c0694c9834 
  third_party/libprocess/src/tests.cpp 1df7ae61f24643a0ceab146646e77da6909dfd5f 

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


Testing
-------

make check on osx, and redhat

mesos-local.sh to verify pailer


Thanks,

Ben Mahler

Reply via email to