Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-1915 Multiplexing Services
  3. THRIFT-1902

C++: Support for Multiplexing Services on any Transport, Protocol and Server

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.9
    • Fix Version/s: 0.9.1
    • Component/s: C++ - Library
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      Make patch available for C++ implementation of the Java TMultiplexedProtocol and related classes as described in THRIFT-563.

      1. multiplex_c++.patch
        30 kB
        Patrik Lindblom

        Issue Links

          Activity

          Hide
          linpatrik Patrik Lindblom added a comment -

          Created a patch including C++ implementation of TMultiplexedProtocol, TProtocolDecorator, and TMultiplexedProcessor classes.
          The patch is based on thrift-0.9.0

          Show
          linpatrik Patrik Lindblom added a comment - Created a patch including C++ implementation of TMultiplexedProtocol, TProtocolDecorator, and TMultiplexedProcessor classes. The patch is based on thrift-0.9.0
          Hide
          roger.meier Roger Meier added a comment -

          Cool!

          I committed this.
          However, I leave the ticket open and wait for a test suite ;-r

          I would love to see an integrated test in our cross language test suite started here:

          sh bootstrap.sh
          ./configure
          make check
          sh test/test.sh
          
          Show
          roger.meier Roger Meier added a comment - Cool! I committed this. However, I leave the ticket open and wait for a test suite ;-r I would love to see an integrated test in our cross language test suite started here: sh bootstrap.sh ./configure make check sh test/test.sh
          Hide
          hudson Hudson added a comment -

          Integrated in Thrift #635 (See https://builds.apache.org/job/Thrift/635/)
          THRIFT-1902 C++: Support for Multiplexing Services on any Transport, Protocol and Server (Revision 3a931b50d906db8099d3819a4c8c89328bb92a61)

          Result = FAILURE
          roger : https://git-wip-us.apache.org/repos/asf?p=thrift.git&a=commit&h=3a931b50d906db8099d3819a4c8c89328bb92a61
          Files :

          • lib/cpp/src/thrift/protocol/TMultiplexedProtocol.h
          • lib/cpp/src/thrift/protocol/TMultiplexedProtocol.cpp
          • lib/cpp/Makefile.am
          • lib/cpp/src/thrift/protocol/TProtocolDecorator.h
          • lib/cpp/src/thrift/processor/TMultiplexedProcessor.h
          Show
          hudson Hudson added a comment - Integrated in Thrift #635 (See https://builds.apache.org/job/Thrift/635/ ) THRIFT-1902 C++: Support for Multiplexing Services on any Transport, Protocol and Server (Revision 3a931b50d906db8099d3819a4c8c89328bb92a61) Result = FAILURE roger : https://git-wip-us.apache.org/repos/asf?p=thrift.git&a=commit&h=3a931b50d906db8099d3819a4c8c89328bb92a61 Files : lib/cpp/src/thrift/protocol/TMultiplexedProtocol.h lib/cpp/src/thrift/protocol/TMultiplexedProtocol.cpp lib/cpp/Makefile.am lib/cpp/src/thrift/protocol/TProtocolDecorator.h lib/cpp/src/thrift/processor/TMultiplexedProcessor.h
          Hide
          jensg Jens Geyer added a comment -

          I created THRIFT-2013 for the test integration, as this part of the issue belongs to all languages.

          => Resolved this one.

          Show
          jensg Jens Geyer added a comment - I created THRIFT-2013 for the test integration, as this part of the issue belongs to all languages. => Resolved this one.

            People

            • Assignee:
              Unassigned
              Reporter:
              linpatrik Patrik Lindblom
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development