> On June 30, 2015, 4:24 p.m., Anand Mazumdar wrote: > > 3rdparty/libprocess/src/http.cpp, line 777 > > <https://reviews.apache.org/r/36040/diff/1/?file=995839#file995839line777> > > > > What is this change intending to solve ? If it's the ability to not > > close the connection for streamed responses. > > > > We would only like to not close the connection i.e. communicate to the > > client that it's a persistent connection only when using the streaming > > API's. > > > > So the condition should be : > > > > if ( streamedResponse ) { > > ... // don't send the close header > > } > > > > In all other cases, we don't want to use the same connection again, and > > it's fine to send the "close" header like we are doing. > > > > Can you also add @bmahler as a reviewer to confirm this ?
I mostly add this for tests on 36037, this overrides the conenction header otherwise. - Isabel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36040/#review89909 ----------------------------------------------------------- On June 30, 2015, 9:06 a.m., Isabel Jimenez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36040/ > ----------------------------------------------------------- > > (Updated June 30, 2015, 9:06 a.m.) > > > Review request for mesos, Anand Mazumdar, Marco Massenzio, and Vinod Kone. > > > Repository: mesos-incubating > > > Description > ------- > > Adding the possibility to 'keep-alive' the connection > > > Diffs > ----- > > 3rdparty/libprocess/src/http.cpp 0898335 > > Diff: https://reviews.apache.org/r/36040/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Isabel Jimenez > >
