Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-4578

Move `TAsyncProtocolProcessor` into main thrift library

VotersStop watchingWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.11.0
    • Fix Version/s: 0.12.0
    • Component/s: C++ - Library
    • Labels:
      None

      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.

        Attachments

          Activity

            People

            • Assignee:
              jking3 James E. King III
              Reporter:
              stefbo Stefan Bolus

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment