mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <bmah...@apache.org>
Subject Re: Review Request 61410: Remove support for omitting 202 responses to old libprocess clients.
Date Fri, 04 Aug 2017 02:38:22 GMT


> On Aug. 3, 2017, 7:13 p.m., Benjamin Hindman wrote:
> > Random question, do you know if there is a test that sends responses back to libprocess
to exercise the `process::internal::ignore_recv_data` function?

Hm.. I couldn't find one.


> On Aug. 3, 2017, 7:13 p.m., Benjamin Hindman wrote:
> > 3rdparty/libprocess/src/tests/process_tests.cpp
> > Lines 1318 (patched)
> > <https://reviews.apache.org/r/61410/diff/1/?file=1789074#file1789074line1318>
> >
> >     Were these actually necessary? I thought this was only necessary after we started
to use `http::Server` since without `keepAlive` the `http::Server` will shutdown the socket
right away (and `connection.disconnect()` would fail?).

They're necessary because before this change no response is received, so the connection remains
open even with no keep alive. After this change, the response is received and http::Connection
will disconnect after receiving the response. This test expects to be able to disconnect sucessfully,
whereas disconnect() fails if already disconnected (although perhaps it should just return
a ready future in that case!)


- Benjamin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61410/#review182140
-----------------------------------------------------------


On Aug. 3, 2017, 6:30 p.m., Benjamin Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61410/
> -----------------------------------------------------------
> 
> (Updated Aug. 3, 2017, 6:30 p.m.)
> 
> 
> Review request for mesos and Benjamin Hindman.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Prior to commit d5fe51c on April 11 2014, we needed to omit the 202
> responses for libprocess messages because libprocess did not read
> the data from its message passing sockets.
> 
> This change removes support for omitting the responses to old clients.
> This also means that any 3rdparty libprocess clients (e.g. someone's
> go library) need to be correctly reading from their sockets to
> communicate with libprocess after this change.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/src/process.cpp af5a75950bf24059d291acfd48399ab2d55eb8c2 
>   3rdparty/libprocess/src/tests/process_tests.cpp 30d0fb845468ff993e42a5568f57be131f0cd24b

> 
> 
> Diff: https://reviews.apache.org/r/61410/diff/1/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Benjamin Mahler
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message