GitHub user gadLinux opened a pull request:
https://github.com/apache/thrift/pull/927
THRIFT-3706 : Implement protocol multiplexor in c_glib [Rebased Master]
This patch implements protocol multiplexor for the client. For the server
there's a need for a processor that currently we don't have. So a test of this
can only be tested against other language that already has this in place. We
did testing against Java.
Example of use:
/*
We will use binary encoding
*/
ThriftBinaryProtocol *binary_protocol= g_object_new
(THRIFT_TYPE_BINARY_PROTOCOL,
"transport", manager->transport,
NULL);
/*
FIXME Transport in the multiplexed should be got from binary_protocol
*/
ThriftMultiplexedProtocol *multiplexed_protocol= g_object_new
(THRIFT_TYPE_MULTIPLEXED_PROTOCOL,
"transport", manager->transport,
"protocol", binary_protocol,
"service-name", "MyService",
NULL);
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gadLinux/thrift thrift-3706
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/927.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 #927
----
commit a73022a5496122a7f3bd20b9c15ab699852f5239
Author: Gonzalo Aguilar Delgado <[email protected]>
Date: 2016-03-04T12:16:22Z
Implement multiplexed protocol
commit a5a6ef17af5ff10187ed926956a30155635203c0
Author: Gonzalo Aguilar Delgado <[email protected]>
Date: 2016-03-05T00:21:27Z
Implement protocol multiplexor. For me there are a pair of things missing.
Implement protocol decorator.
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
