[THRIFT-4578] Move `TAsyncProtocolProcessor` into main thrift library

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.11.0
• Fix Version/s: 0.12.0
• Component/s: C++ - Library
• Assignee: James E. King III
• Reporter: Stefan Bolus
• Created: 05/Jun/18 19:21
• Updated: 27/Dec/18 15:24
• Resolved: 07/Jun/18 13:46

Description

Currently, when using asynchronous `cob_style` servers with Thrift, linking the `thriftnb` library is mandatory to have the definitions of the class `TAsyncProtocolProcessor`. This implies linking against `libevent` for the Evhttp async. client and server. When using custom transports like ZeroMQ, `libevent` is not necessary and thus, using it should be optional.

The easiest way so solve the issues seems to move `TAsyncProtocolProcessor` to the `thrift` library itself. At least for the server side. I haven't checked the client side yet.

Issue Links

• links to: GitHub Pull Request #1562