Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-8214

Remove proxy and engine, rely directly on pb generated Service

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.95.1
    • IPC/RPC
    • None
    • Reviewed
    • Take on the protobuf Service model. Undoes two layers of rpc -- a proxy that did reflection to hook up methods and an 'engine' layer that was put in place to support secure/insecure and Writables vs Protobuf rpc'ing.
    • 0.96notable

    Description

      Attached patch is not done. Removes two to three layers – depending on how you count – between client and rpc client and similar on server side (between rpc and server implementation). Strips ProtobufRpcServer/Client and HBaseClientRpc/HBaseServerRpc. Also gets rid of proxy.

      Attachments

        1. 8124.txt
          525 kB
          Michael Stack
        2. 8214v10.txt
          790 kB
          Michael Stack
        3. 8214v11.txt
          776 kB
          Michael Stack
        4. 8214v12.txt
          776 kB
          Michael Stack
        5. 8214v12 (1).txt
          776 kB
          Michael Stack
        6. 8214v15.txt
          778 kB
          Michael Stack
        7. 8214v15.txt
          778 kB
          Michael Stack
        8. 8214v15.txt
          778 kB
          Michael Stack
        9. 8214v2.txt
          687 kB
          Michael Stack
        10. 8214v3.txt
          689 kB
          Michael Stack
        11. 8214v4.txt
          690 kB
          Michael Stack
        12. 8214v5.txt
          694 kB
          Michael Stack
        13. 8214v6.txt
          690 kB
          Michael Stack
        14. 8214v7.txt
          767 kB
          Michael Stack
        15. 8214v8.txt
          773 kB
          Michael Stack
        16. 8214v8-nosecinfo.patch
          72 kB
          Gary Helmling
        17. 8214v9.txt
          778 kB
          Michael Stack

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            stack Michael Stack
            stack Michael Stack
            Votes:
            0 Vote for this issue
            Watchers:
            15 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment