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

Consolidate TSimpleServer, TThreadedServer, TThreadPoolServer into one

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Won't Fix
    • 0.9.2, 0.9.3
    • None
    • C++ - Library
    • None

    Description

      It was suggested in discussions inside THRIFT-3083, THRIFT-3084 that the difference between the refactored servers was so slight that we might want to start thinking about them as one server. We have IO Completion servers, Non Blocking servers, and threaded servers with concurrency control (1 = Simple, >1 = Threaded/ThreadPool). Each one brings a substantially different mechanism to the table and are valid to stand on their own, but maybe it is time to consolidate into a single TThreadedServer.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. THRIFT-3768 TThreadedServer may crash if it is destroyed immediately after it returns from serve(); TThreadedServer disconnects clients when they connec

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. THRIFT-3081 C++ Consolidate client processing loops in TServers

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. THRIFT-3083 C++ Consolidate server processing loops in TSimpleServer, TThreadedServer, TThreadPoolServer

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. THRIFT-3084 C++ add concurrent client limit to threaded servers

          • Major - Major loss of function.
          • Closed

          Activity

            People

              jking3 James E. King III
              jking3 James E. King III
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: