GitHub user RobberPhex opened a pull request:
https://github.com/apache/thrift/pull/1022
THRIFT-3845
use `isBinaryAccelerated` to indicate use extension `thrift_protocol`.
When `TBinaryProtocolAccelerated` warped in
`TMultiplexedProtocol`,extension `thrift_protocol` can also be used for `client
serialization`, `client deserialization`, `server serialization`.
notice: `server deserialization` can't use extension `thrift_protocol`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/RobberPhex/thrift THRIFT-3845
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1022.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 #1022
----
commit 30d3fb6000f84d401884a7799f2a96a08930e917
Author: Robert Lu <[email protected]>
Date: 2016-05-31T15:38:18Z
add isStrict{Read,Write} to TProtocolDecorator for
TBinaryProtocolAccelerated
commit 2b9005a49c299ab444a21d61e38b1714f294f5cf
Author: Robert Lu <[email protected]>
Date: 2016-05-31T16:39:06Z
add isBinaryAccelerated for thrift_protocol ext
instanceof TBinaryProtocolAccelerated will caused
TBinaryProtocolAccelerated warped in TMultiplexedProtocol can't use
thrift_protocol extension
----
---
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.
---
