[jira] [Commented] (THRIFT-4578) Move `TAsyncProtocolProcessor` into main thrift library

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/THRIFT-4578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16503737#comment-16503737 ] 

ASF GitHub Bot commented on THRIFT-4578:
----------------------------------------

stefbo opened a new pull request #1562: THRIFT-4578 Move `TAsyncProtocolProcessor` into main thrift library
URL: https://github.com/apache/thrift/pull/1562
 
 
   Updated CMake and automake files for C++.

----------------------------------------------------------------
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:
users@infra.apache.org


> Move `TAsyncProtocolProcessor` into main thrift library
> -------------------------------------------------------
>
>                 Key: THRIFT-4578
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4578
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>    Affects Versions: 0.11.0
>            Reporter: Stefan Bolus
>            Priority: Minor
>
> 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.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)