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

Cleanup CoordinatedStateManager

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

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-alpha-4, 2.0.0
    • None
    • None

    Description

      • Remove the configuration hbase.coordinated.state.manager.class
      • Keep following interface since they nicely separate ZK based
        implementation: SplitLogWorkerCoordination, SplitLogManagerCoordination,
        ProcedureCoordinatorRpcs, ProcedureMemberRpcs
      • Replace CSM (interface) + BCSM (unnecessary middle hierarchy) with single
        CSM interface.
      • Don't pass whole CSM object around (with server in it which gives acess to pretty much everything), only pass the relevant dependencies.

      Discussion thread on dev@ mailing list.
      http://mail-archives.apache.org/mod_mbox/hbase-dev/201710.mbox/%3CCAAjhxrqjOg90Fdi73kZZe_Gxtrqq8ff%2B%3DAj_epptO_XO812Abg%40mail.gmail.com%3E

      Attachments

        1. HBASE-19073.master.001.patch
          85 kB
          Apekshit Sharma
        2. HBASE-19073.master.002.patch
          86 kB
          Apekshit Sharma

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

          People

            appy Apekshit Sharma
            appy Apekshit Sharma
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment