Thrift
  1. Thrift
  2. THRIFT-1915 Multiplexing Services
  3. THRIFT-2144

Cocoa: Client-side support for Multiplexing Services on any Transport and Protocol

    Details

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

      Description

      The attached patch provides client-side support for multiplexed services as specified at THRIFT-563.

        Issue Links

          Activity

          Hide
          ASF GitHub Bot added a comment -

          Github user drobakowski commented on the pull request:

          https://github.com/apache/thrift/pull/150#issuecomment-47987834

          Hi @guoyalun,

          before you create a new JIRA ticket, i think this duplicates my ticket THRIFT-2144(https://issues.apache.org/jira/browse/THRIFT-2144) as well as parts of the unmerged PR #49. I'm currently rebasing my closed and unmerged PRs.

          Greets,
          David

          Show
          ASF GitHub Bot added a comment - Github user drobakowski commented on the pull request: https://github.com/apache/thrift/pull/150#issuecomment-47987834 Hi @guoyalun, before you create a new JIRA ticket, i think this duplicates my ticket THRIFT-2144 ( https://issues.apache.org/jira/browse/THRIFT-2144 ) as well as parts of the unmerged PR #49. I'm currently rebasing my closed and unmerged PRs. Greets, David
          Hide
          Jens Geyer added a comment -

          True, didn't remember this one. Thanks David.

          Show
          Jens Geyer added a comment - True, didn't remember this one. Thanks David.
          Hide
          ASF GitHub Bot added a comment -

          GitHub user drobakowski opened a pull request:

          https://github.com/apache/thrift/pull/151

          THRIFT-2144 - Cocoa: Client-side support for Multiplexing Services on any Transport and Protocol

          This pull request provides Cocoa client-side support for multiplexed services as specified at THRIFT-563. Partly extracted from my initial PR https://github.com/apache/thrift/pull/49.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/synlay/thrift THRIFT-2144

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/thrift/pull/151.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #151


          commit 1b6cbda93f10fd38d1e893c17c84d4baedf51f06
          Author: David Robakowski <david.robakowski@synlay.com>
          Date: 2014-07-03T21:26:07Z

          Implement Cocoa client-side support for Multiplexing Services
          on any Transport and Protocol as described
          here: https://issues.apache.org/jira/browse/THRIFT-563


          Show
          ASF GitHub Bot added a comment - GitHub user drobakowski opened a pull request: https://github.com/apache/thrift/pull/151 THRIFT-2144 - Cocoa: Client-side support for Multiplexing Services on any Transport and Protocol This pull request provides Cocoa client-side support for multiplexed services as specified at THRIFT-563 . Partly extracted from my initial PR https://github.com/apache/thrift/pull/49 . You can merge this pull request into a Git repository by running: $ git pull https://github.com/synlay/thrift THRIFT-2144 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/151.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #151 commit 1b6cbda93f10fd38d1e893c17c84d4baedf51f06 Author: David Robakowski <david.robakowski@synlay.com> Date: 2014-07-03T21:26:07Z Implement Cocoa client-side support for Multiplexing Services on any Transport and Protocol as described here: https://issues.apache.org/jira/browse/THRIFT-563
          Hide
          David Robakowski added a comment -

          Jens Geyer no problem! I removed the initial patch files from this commit to keep better track of possible feature work/rebases. I will also create a new PR with the tutorial updates for this ticket as soon as this ticket gets closed.

          Show
          David Robakowski added a comment - Jens Geyer no problem! I removed the initial patch files from this commit to keep better track of possible feature work/rebases. I will also create a new PR with the tutorial updates for this ticket as soon as this ticket gets closed.
          Hide
          ASF GitHub Bot added a comment -

          Github user drobakowski commented on the pull request:

          https://github.com/apache/thrift/pull/151#issuecomment-48805568

          see #150

          Show
          ASF GitHub Bot added a comment - Github user drobakowski commented on the pull request: https://github.com/apache/thrift/pull/151#issuecomment-48805568 see #150
          Hide
          Jens Geyer added a comment -

          Hi, whats the state of this ticket? Is it ready to be reviewed and merged?

          Show
          Jens Geyer added a comment - Hi, whats the state of this ticket? Is it ready to be reviewed and merged?
          Hide
          David Robakowski added a comment -

          Hi Jens Geyer, I think it's ready to be reviewed. I've already tried to start a discussion for one or the other patch with guoyalun, but unfortunately he didn't respond so far. Please see my issue comment on GitHub.

          Show
          David Robakowski added a comment - Hi Jens Geyer , I think it's ready to be reviewed. I've already tried to start a discussion for one or the other patch with guoyalun, but unfortunately he didn't respond so far. Please see my issue comment on GitHub.
          Hide
          ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/thrift/pull/151

          Show
          ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/151
          Hide
          Jens Geyer added a comment -

          Committed, thanks!

          Show
          Jens Geyer added a comment - Committed, thanks!
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Thrift-Test #31 (See https://builds.apache.org/job/Thrift-Test/31/)
          THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75)

          • lib/cocoa/src/protocol/TProtocolDecorator.h
          • lib/cocoa/src/protocol/TMultiplexedProtocol.m
          • lib/cocoa/src/protocol/TMultiplexedProtocol.h
          • lib/cocoa/src/protocol/TProtocolDecorator.m
          Show
          Hudson added a comment - FAILURE: Integrated in Thrift-Test #31 (See https://builds.apache.org/job/Thrift-Test/31/ ) THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75) lib/cocoa/src/protocol/TProtocolDecorator.h lib/cocoa/src/protocol/TMultiplexedProtocol.m lib/cocoa/src/protocol/TMultiplexedProtocol.h lib/cocoa/src/protocol/TProtocolDecorator.m
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Thrift #1286 (See https://builds.apache.org/job/Thrift/1286/)
          THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75)

          • lib/cocoa/src/protocol/TProtocolDecorator.h
          • lib/cocoa/src/protocol/TProtocolDecorator.m
          • lib/cocoa/src/protocol/TMultiplexedProtocol.m
          • lib/cocoa/src/protocol/TMultiplexedProtocol.h
          Show
          Hudson added a comment - SUCCESS: Integrated in Thrift #1286 (See https://builds.apache.org/job/Thrift/1286/ ) THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75) lib/cocoa/src/protocol/TProtocolDecorator.h lib/cocoa/src/protocol/TProtocolDecorator.m lib/cocoa/src/protocol/TMultiplexedProtocol.m lib/cocoa/src/protocol/TMultiplexedProtocol.h
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Thrift #1331 (See https://builds.apache.org/job/Thrift/1331/)
          THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75)

          • lib/cocoa/src/protocol/TProtocolDecorator.m
          • lib/cocoa/src/protocol/TMultiplexedProtocol.m
          • lib/cocoa/src/protocol/TMultiplexedProtocol.h
          • lib/cocoa/src/protocol/TProtocolDecorator.h
          Show
          Hudson added a comment - SUCCESS: Integrated in Thrift #1331 (See https://builds.apache.org/job/Thrift/1331/ ) THRIFT-2144 Client-side support for Multiplexing Services on any Transport and Protocol (jensg: rev aa99e0e2b8b29b75b18b5dfc8e8c17245cb43e75) lib/cocoa/src/protocol/TProtocolDecorator.m lib/cocoa/src/protocol/TMultiplexedProtocol.m lib/cocoa/src/protocol/TMultiplexedProtocol.h lib/cocoa/src/protocol/TProtocolDecorator.h

            People

            • Assignee:
              Jens Geyer
              Reporter:
              David Robakowski
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development