You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by oprudkyi <gi...@git.apache.org> on 2017/10/29 16:03:41 UTC

[GitHub] thrift pull request #1407: THRIFT-4377: close socket after processing reques...

GitHub user oprudkyi opened a pull request:

    https://github.com/apache/thrift/pull/1407

    THRIFT-4377: close socket after processing request

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/oprudkyi/thrift THRIFT-4377

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/1407.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 #1407
    
----
commit 42b29a9a4e161c1261be22ef826bea95ac5b5251
Author: Oleksii Prudkyi <ol...@gmail.com>
Date:   2017-10-29T16:01:14Z

    THRIFT-4377: close socket after processing request

----


---

[GitHub] thrift issue #1407: THRIFT-4377: close socket after processing request

Posted by oprudkyi <gi...@git.apache.org>.

Github user oprudkyi commented on the issue:

    https://github.com/apache/thrift/pull/1407
  
    @jeking3 , Sorry, there was my fault - haxe php integration was broken by first commit and I added fixes 


---

[GitHub] thrift pull request #1407: THRIFT-4377: close socket after processing reques...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/thrift/pull/1407


---

[GitHub] thrift issue #1407: THRIFT-4377: close socket after processing request

Posted by jeking3 <gi...@git.apache.org>.

Github user jeking3 commented on the issue:

    https://github.com/apache/thrift/pull/1407
  
    I wonder if this is why sleep 10 is needed between haxe tests and whether that could be improved now?  I'll apply the other ones and see. :)


---

[GitHub] thrift issue #1407: THRIFT-4377: close socket after processing request

Posted by jeking3 <gi...@git.apache.org>.

Github user jeking3 commented on the issue:

    https://github.com/apache/thrift/pull/1407
  
    @Jens-G I see only one of the three commits here committed in master?


---