GitHub user chetmurthy opened a pull request:
https://github.com/apache/thrift/pull/1418
Thrift 3877 http server buffering bug oneway
This PR is in two commits, so that the reviewer can
(a) apply the testcase commit, elicit the failure by running
lib/cpp/tests/UnitTests
(b) apply the (full) second commit, rerun testcase, and see the problem is
gone.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/chetmurthy/thrift
THRIFT-3877-http-server-buffering-bug-oneway
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1418.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1418
----
commit 7d83cb4a2cafe033e7d18705cc0f74cd9b883cd5
Author: Chet Murthy <[email protected]>
Date: 2017-11-27T00:39:15Z
THRIFT-3877 unit-test elicits failure of C++/HTTP/oneway hang
Hit a C++ HTTP server with a oneway rpc, and the next RPC will hang.
This test-case elicits the failure (converts to timeout-expiry).
commit 10c8951de5f483b9bc71e2aa1558dd0170389bce
Author: Chet Murthy <[email protected]>
Date: 2017-11-27T00:42:28Z
THRIFT-3877 this fixes the subject bug
C++ HTTP server, hit with oneway RPC, then roundtrip RPC, no longer
hangs, as demonstrated by OneWayHTTPTest.
----
---
