VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Blocker
    • Resolution: Implemented
    • 2.0.0
    • 2.0.0-alpha-4, 2.0.0
    • Coprocessors
    • None
    • Remove bypass from all Master operations.

    Description

      Lets decide on this one before we release 2.0.0.

      A bunch of methods in MasterCoprocessorHost on the 'pre' step allow returning true which indicates the method invocation is not to proceed.

      Not all 'pre' steps do this. Just some.

      Seems a little arbitrary.

      How we skip out if we are not proceed with the invocation is also a little arbitrary.

      When a deleteColumn call is supposed to skip out, it returns a -1, a non-procId. If we are to skip a balance call, we log that CP said skip and then return false to indicate the balancer did not run (why?). Elsewhere we just exit silently. In createNamespace we used to exit silently but HBASE-14888 just changed it so we throw a BypassCoprocessorException instead...

      Lets make them all work the same way.

      (This issue comes of chat w/ Matteo)

      Attachments

        1. HBASE-15071.patch
          29 kB
          Heng Chen

        Issue Links

        Activity

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

          People

            chenheng Heng Chen
            stack Michael Stack
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment