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

Posted to dev@thrift.apache.org by Jens-G <gi...@git.apache.org> on 2016/07/24 10:20:59 UTC

[GitHub] thrift pull request #1055: THRIFT-2156

GitHub user Jens-G opened a pull request:

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

    THRIFT-2156 

    code from pull request #1033 plus some refactoring added

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

    $ git pull https://github.com/Jens-G/thrift THRIFT-2156

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

    https://github.com/apache/thrift/pull/1055.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 #1055
    
----
commit 2988bdd14fc4f4602a58407929e331a4649a60c5
Author: Jim King <ji...@simplivity.com>
Date:   2016-06-20T05:08:58Z

    THRIFT-2156: fix errno handling in server socket

commit 4bf852d6747c08e1e1b8b5bc16cc9481ad63bf56
Author: Jens Geyer <je...@apache.org>
Date:   2016-07-24T10:11:25Z

    THRIFT-2156: fix errno handling in server socket

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] thrift issue #1055: THRIFT-2156

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

Github user jeking3 commented on the issue:

    https://github.com/apache/thrift/pull/1055
  
    Agree :+1:


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] thrift issue #1055: THRIFT-2156

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

Github user tpcwang commented on the issue:

    https://github.com/apache/thrift/pull/1055
  
    Looks good :+1: 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] thrift issue #1055: THRIFT-2156

Posted by Jens-G <gi...@git.apache.org>.

Github user Jens-G commented on the issue:

    https://github.com/apache/thrift/pull/1055
  
    @jeking + @tpcwang please feel free to comment


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] thrift pull request #1055: THRIFT-2156

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

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---