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

Move `TAsyncProtocolProcessor` into main thrift library

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 0.11.0
    • 0.12.0
    • C++ - Library
    • 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

        Issue Links

          Activity

            People

              jking3 James E. King III
              stefbo Stefan Bolus
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: