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

Remove CoprocessorProtocol support and implementations

    XMLWordPrintableJSON

Details

    • Task
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • 0.95.2
    • 0.95.0
    • Coprocessors
    • None
    • Incompatible change, Reviewed
    • Hide
      Dynamic endpoints must now be refactored to use CoprocessorService instead of CoprocessorProtocol. See the coprocessor package info javadoc for example. Study unit tests for examples where we migrate for old style to new. Write the mailing list if you need help because a few fellas have volunteered to help those who need to migrate.
      Show
      Dynamic endpoints must now be refactored to use CoprocessorService instead of CoprocessorProtocol. See the coprocessor package info javadoc for example. Study unit tests for examples where we migrate for old style to new. Write the mailing list if you need help because a few fellas have volunteered to help those who need to migrate.

    Description

      The CoprocessorProtocol mechanism for registering and calling coprocessor endpoints has been supplanted by protocol buffer based service registrations in HBASE-5448. This means the Writable-based CoprocessorProtocol mechanism will need to be removed at some point.

      In discussion on the dev list, general consensus was that it is better to remove the CoprocessorProtocol mechanism completely in 0.96, where other backwards-incompatible changes are being made, and before it is adopted by more clients:
      http://mail-archives.apache.org/mod_mbox/hbase-dev/201209.mbox/%3CCADfYSXTvqEu3SifyWdZCi9kKPw1Hf9EoaKVGFOCbwRpSkdaCng%40mail.gmail.com%3E

      This is an umbrella issue to track what changes are required to completely remove the CoprocessorProtocol support.

      Attachments

        1. 6895v2.txt
          206 kB
          Michael Stack
        2. 6895.txt
          206 kB
          Michael Stack

        Issue Links

          Activity

            People

              stack Michael Stack
              ghelmling Gary Helmling
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: