----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32347/#review77967 -----------------------------------------------------------
Ship it! 3rdparty/libprocess/src/decoder.hpp <https://reviews.apache.org/r/32347/#comment126336> Looks like we won't be using the old ResponseDecoder anymore after https://reviews.apache.org/r/32351. Are you planing to kill the old response decoder? If yes, the name "Streaming" here seems to be uncessary once the old response decoder is killed. Are you going to s/StreamingResponseDecoder/ResponseDecoder/ after r32351 is committed? 3rdparty/libprocess/src/decoder.hpp <https://reviews.apache.org/r/32347/#comment126346> ``` CHECK_NOTNULL(decoder->response); ``` ? 3rdparty/libprocess/src/tests/decoder_tests.cpp <https://reviews.apache.org/r/32347/#comment126372> Do you want to move this up right after ``` decoder.decode(body.data(), body.length()); ``` ? - Jie Yu On March 20, 2015, 11:51 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32347/ > ----------------------------------------------------------- > > (Updated March 20, 2015, 11:51 p.m.) > > > Review request for mesos, Benjamin Hindman and Jie Yu. > > > Bugs: MESOS-2438 > https://issues.apache.org/jira/browse/MESOS-2438 > > > Repository: mesos > > > Description > ------- > > Provides a response decoder that returns 'PIPE' responses once the response > headers are received, but before the body data is received. Callers are > expected to read the body from the Pipe::Reader in the response. > > This is needed to receive streaming responses on the client-side. > > > Diffs > ----- > > 3rdparty/libprocess/src/decoder.hpp > 45f3d7fff2c33ce9c37a8ecbdcbf2c0f02ee76e8 > 3rdparty/libprocess/src/tests/decoder_tests.cpp > d65f5cf25eaecd5994404ccdf07f32d703a5955b > > Diff: https://reviews.apache.org/r/32347/diff/ > > > Testing > ------- > > Added a test. > > > Thanks, > > Ben Mahler > >