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

Purge 'complete' support from Coprocesor Observers

    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-beta-1, 2.0.0
    • Coprocessors
    • None
    • Reviewed
    • This issue removes the 'complete' facility that was in ObserverContext. It is no longer possible for a Coprocessor to cut the chain-of-invocation and insist its response prevails.

    Description

      Up on dev list under '[DISCUSSION] Removing the bypass semantic from the Coprocessor APIs', we are discussing purge of 'complete'. Unless objection, lets purge for beta-1.

      andrew.purtell@gmail.com says the following up on the dev list:

      It would simplify the theory of operation for coprocessors if we can assume either the entire chain will complete or one of the coprocessors in the chain will throw an exception that not only terminates processing of the rest of the chain but also the operation in progress.
      Security coprocessors interrupt processing by throwing an exception, which is meant to propagate all the way back to the user.
      I think it's more than fair to ask the same question about 'complete' as we did about 'bypass': Does anyone use it? Is it needed?

      Attachments

        1. HBASE-19123.master.005.patch
          16 kB
          Michael Stack
        2. HBASE-19123.master.004.patch
          19 kB
          Michael Stack
        3. HBASE-19123.master.003.patch
          19 kB
          Michael Stack
        4. HBASE-19123.master.002.patch
          18 kB
          Michael Stack
        5. HBASE-19123.master.001.patch
          18 kB
          Michael Stack

        Issue Links

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: