Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-17856 Further improvements for asynchronous hbase client
  3. HBASE-22040

Add mergeRegionsAsync with a List of region names method in AsyncAdmin

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Reviewed
    • Hide
      Add a mergeRegionsAsync(byte[][], boolean) method in the AsyncAdmin interface.

      Instead of using assert, now we will throw IllegalArgumentException when you want to merge less than 2 regions at client side. And also, at master side, instead of using assert, now we will throw DoNotRetryIOException if you want merge more than 2 regions, since we only support merging two regions at once for now.
      Show
      Add a mergeRegionsAsync(byte[][], boolean) method in the AsyncAdmin interface. Instead of using assert, now we will throw IllegalArgumentException when you want to merge less than 2 regions at client side. And also, at master side, instead of using assert, now we will throw DoNotRetryIOException if you want merge more than 2 regions, since we only support merging two regions at once for now.

    Description

      Although we only support merging two regions until now, but the rpc protocol does support passing more than two regions to master.

      So I think we should provide the methods, but need to add comments to say that for now we only support merging two regions so do not try to pass more than two regions.

      Attachments

        1. HBASE-22040.patch
          32 kB
          Duo Zhang
        2. HBASE-22040-branch-2.patch
          19 kB
          Duo Zhang
        3. HBASE-22040-v1.patch
          19 kB
          Duo Zhang
        4. HBASE-22040-v2.patch
          19 kB
          Duo Zhang
        5. HBASE-22040-v3.patch
          19 kB
          Duo Zhang

        Issue Links

        Activity

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

          People

            zhangduo Duo Zhang
            zhangduo Duo Zhang
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment