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

Polish the Admin interface

    XMLWordPrintableJSON

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.0.0-alpha-1, 2.3.0
    • Admin, Client
    • None
    • Incompatible change, Reviewed
    • Hide
      Add a cloneSnapshotAsync method with restoreAcl parameter.
      Deprecated restoreSnapshotAsync method as it just ignores the failsafe configuration.
      Make snapshotAsync method returns a Future<Void>.
      Deprecated the snapshot related methods which take a 'byte[]' as the snapshot name.
      Use default methods to reduce the code base for implementation classes.
      Show
      Add a cloneSnapshotAsync method with restoreAcl parameter. Deprecated restoreSnapshotAsync method as it just ignores the failsafe configuration. Make snapshotAsync method returns a Future<Void>. Deprecated the snapshot related methods which take a 'byte[]' as the snapshot name. Use default methods to reduce the code base for implementation classes.

    Description

      The snapshot related methods are not well declared, we missed several methods which has the restoreAcl parameter. And also, the snapshotAsync method returns nothing, which is bit strange.

      And we can use default methods to reduce the code in HBaseAdmin and also the new Admin implementation in the future.

      Attachments

        1. HBASE-22001.patch
          82 kB
          Duo Zhang
        2. HBASE-22001-v1.patch
          85 kB
          Duo Zhang
        3. HBASE-22001-v2.patch
          85 kB
          Duo Zhang
        4. HBASE-22001-v3.patch
          86 kB
          Duo Zhang
        5. HBASE-22001-v3.patch
          86 kB
          Duo Zhang
        6. HBASE-22001-v4.patch
          86 kB
          Duo Zhang
        7. HBASE-22001-v4.patch
          86 kB
          Duo Zhang
        8. HBASE-22001-v5.patch
          86 kB
          Duo Zhang

        Issue Links

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: