Re: Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

2015-03-30 Thread Ben Mahler
> On March 30, 2015, 6:40 p.m., Jie Yu wrote: > > 3rdparty/libprocess/include/process/http.hpp, line 587 > > > > > > Can you put a TODO here you wrote in the description of this review: > > ``` > > Long term I'

Re: Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

2015-03-30 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32351/#review78111 --- Ship it! 3rdparty/libprocess/include/process/http.hpp

Re: Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

2015-03-23 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32351/#review77491 --- Patch looks great! Reviews applied: [32336, 32337, 32338, 32339, 32

Re: Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

2015-03-23 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32351/ --- (Updated March 23, 2015, 11 p.m.) Review request for mesos, Benjamin Hindman, J

Review Request 32351: Added http::streaming::get/post for client-side streaming responses.

2015-03-20 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32351/ --- Review request for mesos, Benjamin Hindman, Jie Yu, and Joris Van Remoortere. B