----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7048/#review12320 -----------------------------------------------------------
Ship it! lgtm src/tests/files_tests.cpp <https://reviews.apache.org/r/7048/#comment26093> align this with pid - Vinod Kone On Oct. 10, 2012, 9:46 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7048/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2012, 9:46 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > see above > > also: > -removed redundant length field from json > -added missing test for buffered io::read > > > Diffs > ----- > > src/files/files.cpp 6d41bf6649479f83b1f5f56665b555c0c553e713 > src/tests/files_tests.cpp 764d2c7fdec4a21b9698eb4cb9e0673bd7f20e9d > 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 > 1cb26cd1729ce1cf0ed9573d3eb40690705d4b4e > > Diff: https://reviews.apache.org/r/7048/diff/ > > > Testing > ------- > > make check on osx, and redhat > > mesos-local.sh to verify pailer > > > Thanks, > > Ben Mahler > >
