[THRIFT-2911] fix c++ version zeromq transport, the old version cannot work

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.9.3
• Component/s: C++ - Library
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Jens Geyer
• Created: 22/Dec/14 21:27
• Updated: 10/May/15 09:54
• Resolved: 22/Dec/14 21:39

Description

GitHub user littletiny opened a pull request:

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

fix c++ version zeromq transport, the old version cannot work

fix c++ version zeromq transport, the old version cannot work
we need to override
```
TTransport::write_virt TTransport::read_virt
```
instead of
```
TTransport::write TTransport::read
```

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

$ git pull https://github.com/littletiny/thrift master

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

https://github.com/apache/thrift/pull/315.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 #315

---

commit bf792bb89cab8e42bbde6e9965bef5c8f2deacd0
Author: tiny <[e-mail]>
Date: 2014-12-11T06:22:29Z

fix c++ version zeromq transport, the old version cannot work

---

Issue Links

• is duplicated by: THRIFT-1649 contrib/zeromq does not build in 0.8.0