You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by gadLinux <gi...@git.apache.org> on 2017/05/25 16:18:26 UTC
[GitHub] thrift pull request #1279: THRIFT-4212: Fix flush on ssl socket thrift
GitHub user gadLinux opened a pull request:
https://github.com/apache/thrift/pull/1279
THRIFT-4212: Fix flush on ssl socket thrift
Fixes the bug in flush on ssl thrift communications transport
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gadLinux/thrift THRIFT-4212
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1279.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 #1279
----
commit 94968a7db10d4b95bec62d33bed66868eebfbc38
Author: Gonzalo Aguilar Delgado <ga...@level2crm.com>
Date: 2017-05-23T15:22:44Z
THRIFT-4205: Make sure gobject+glib are correctly linked
commit fb518bebec7666953fc69a540db9a56973b9bab5
Author: Gonzalo Aguilar Delgado <ga...@level2crm.com>
Date: 2017-05-23T15:24:54Z
Fix lib for glib
commit a813c237fad8997ef96806259b999c5431759caa
Author: Gonzalo Aguilar Delgado <ga...@level2crm.com>
Date: 2017-05-25T15:11:38Z
Fix logging in thrift library in #THRIFT-4211
commit 9250da1fe208927a09539d335ecf01d07e63d082
Author: Gonzalo Aguilar Delgado <ga...@level2crm.com>
Date: 2017-05-25T15:17:37Z
Merge branch 'THRIFT-4211' into RELEASE-1.0.0
commit 84d328d93cab5e51587d892c3ecf641130531c3b
Author: Gonzalo Aguilar Delgado <ga...@level2crm.com>
Date: 2017-05-25T16:15:30Z
Fix flush on invalid 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 #1279: THRIFT-4212: Fix flush on ssl socket thrift
Posted by jeking3 <gi...@git.apache.org>.
Github user jeking3 commented on the issue:
https://github.com/apache/thrift/pull/1279
When THRIFT-4211 is clean in CI and merged, you will need to rebase this since it includes THRIFT-4211, and then we can get it into CI for a build then merge. Thanks.
---
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 #1279: THRIFT-4212: Fix flush on ssl socket thrift
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/thrift/pull/1279
---
[GitHub] thrift issue #1279: THRIFT-4212: Fix flush on ssl socket thrift
Posted by jeking3 <gi...@git.apache.org>.
Github user jeking3 commented on the issue:
https://github.com/apache/thrift/pull/1279
Let's clear up the other two in the pull request backlog, then rebase this and then see what's left.
---