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

Remove HTable references and HTableInterface

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change, Reviewed

      Description

      I want to add a new AsyncTable implementation and to do that with great quality I want to run all the tests with a sync Table implementation which uses the AsyncTable within. To do this the HbaseTestingUtility has to return Table instead of HTable.

      I saw this as the ideal opportunity to remove all deprecated HTableInterface methods in 2.0 branch and thus the interface itself. This means all auto flush settings are removed.

      Some discussion points:

      • I had to remove doBulkLoad(Path hfofDir, final HTable table) from LoadIncrementalHFiles because of the hard HTable usage and replaced usages to the other doBulkLoad(Path hfofDir, final Admin admin, Table table, RegionLocator regionLocator) method.
      • HBaseTestingUtility now returns Table instead of HTable
      • I removed all the methods from HConnection that were also in Connection + I removed the getTable methods which should not be used anymore. (HConnection is deprecated so could be removed itself)

        Attachments

        1. HBASE-15875.v2.patch
          110 kB
          Jurriaan Mous
        2. HBASE-15875.v1.patch
          109 kB
          Jurriaan Mous
        3. HBASE-15875.patch
          122 kB
          Jurriaan Mous

          Issue Links

            Activity

              People

              • Assignee:
                jurmous Jurriaan Mous
                Reporter:
                jurmous Jurriaan Mous
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: