[
https://issues.apache.org/jira/browse/THRIFT-4548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16488576#comment-16488576
]
ASF GitHub Bot commented on THRIFT-4548:
----------------------------------------
cosven commented on issue #1547: THRIFT-4548: python binary accelerated
protocol with multiplexing via decoration
URL: https://github.com/apache/thrift/pull/1547#issuecomment-391602105
> so the decorated protocol object passed into the binary accelerated C
extension was not really a TProtocolBase object, which caused errors.
As far as I know, the C extension does not care what protocol type Python
passed into. It just uses the BinaryProtocol or CompatProtocol define in
bianry/compat.h to do encoding and decoding.
In my opinion, the real problem is that `TXxxProtocol()._fast_encode` is
BuiltInFunction instead of a Method(at the moment). And the
`TProtocolDecorator._wrap` does not handle this case.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Supporting TBinaryProtocolAccelerated protocol when using
> TMultiplexedProcessor in Python
> -----------------------------------------------------------------------------------------
>
> Key: THRIFT-4548
> URL: https://issues.apache.org/jira/browse/THRIFT-4548
> Project: Thrift
> Issue Type: Bug
> Reporter: Balazs Kemenes
> Priority: Major
>
> Supporting TBinaryProtocolAccelerated protocol when using
> TMultiplexedProcessor in Python
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
