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

    XMLWordPrintableJSON

    Details

    • Hadoop Flags:
      Reviewed
    • Release Note:
      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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: