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

Purge 'complete' support from Coprocesor Observers

    XMLWordPrintableJSON

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-beta-1, 2.0.0
    • Component/s: Coprocessors
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      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 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

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

                Dates

                • Created:
                  Updated:
                  Resolved: