----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31930/#review76236 -----------------------------------------------------------
3rdparty/libprocess/include/process/http.hpp <https://reviews.apache.org/r/31930/#comment123821> s/streaming HTTP data/sending an HTTP response via the chunked transfer encoding/ 3rdparty/libprocess/include/process/http.hpp <https://reviews.apache.org/r/31930/#comment123817> Also explain that this is an in-memory single OS process pipe, and is not backed by file descriptors so can't be used across a fork. 3rdparty/libprocess/include/process/http.hpp <https://reviews.apache.org/r/31930/#comment123747> ... Also ignores empty strings as they signify EOF. Which I think is dangerous ultimately ... see Joris review. 3rdparty/libprocess/src/tests/http_tests.cpp <https://reviews.apache.org/r/31930/#comment123819> Let's still keep this as /pipe for now! ;-) Here and below in this test. - Benjamin Hindman On March 11, 2015, 7:41 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31930/ > ----------------------------------------------------------- > > (Updated March 11, 2015, 7:41 a.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Bugs: MESOS-2438 > https://issues.apache.org/jira/browse/MESOS-2438 > > > Repository: mesos > > > Description > ------- > > See MESOS-2438 for the motivation. > > > Diffs > ----- > > 3rdparty/libprocess/include/process/http.hpp > 10143fdbd5eb43f968c41957a2335f96a097d82e > 3rdparty/libprocess/src/http.cpp 7c0cee404645e1dd1a9253f85883aefa29a5f999 > 3rdparty/libprocess/src/process.cpp > e7b029ba97e640c2102548c190ba62b30602f43d > 3rdparty/libprocess/src/tests/http_tests.cpp > 800752a57230d7768d3d741fef87f6283757e424 > > Diff: https://reviews.apache.org/r/31930/diff/ > > > Testing > ------- > > * added tests > * make check > > > Thanks, > > Ben Mahler > >