HBase
  1. HBase
  2. HBASE-359

[hbase] Remove deprecated APIs and change signature of another in HRegionInterface

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      There are a number of deprecated APIs in HRegionInterface (the interface used by the client to communicate with the HRegionServer). Although none of the APIs are used by the current client classes or by other HBase code, we do know that some HBase users have subclassed both the client and HRegionServer. What we do not know is if their changes make use of the deprecated APIs.

      In addition, we would like to make an incompatible change to HRegionInterface.batchUpdate in order to support HADOOP-1724. (It would return either an enum or boolean instead of void).

      If these changes would effect you, please let us know and also give us a time frame when this change could be implemented. Thanks!

      1. patch.txt
        11 kB
        Jim Kellerman

        Issue Links

          Activity

          Hide
          Jim Kellerman added a comment -

          Not going to make additional changes to the region server at this time. Should we later decide to address HADOOP-1724, any corresponding API changes will be handled there.

          Show
          Jim Kellerman added a comment - Not going to make additional changes to the region server at this time. Should we later decide to address HADOOP-1724 , any corresponding API changes will be handled there.
          Hide
          Jim Kellerman added a comment -

          Phase 1 (Remove deprecated APIs) passed tests and were committed.

          Show
          Jim Kellerman added a comment - Phase 1 (Remove deprecated APIs) passed tests and were committed.
          Show
          Hadoop QA added a comment - +1 http://issues.apache.org/jira/secure/attachment/12365201/patch.txt applied and successfully tested against trunk revision r572983. Test results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/692/testReport/ Console output: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/692/console
          Hide
          Jim Kellerman added a comment -

          works locally. try hudson

          Show
          Jim Kellerman added a comment - works locally. try hudson
          Hide
          Jim Kellerman added a comment -

          Phase 1: remove deprecated apis

          Show
          Jim Kellerman added a comment - Phase 1: remove deprecated apis
          Hide
          Jim Kellerman added a comment -

          Last chance. Anyone using the deprecated APIs in HRegionInterface?

          Show
          Jim Kellerman added a comment - Last chance. Anyone using the deprecated APIs in HRegionInterface?

            People

            • Assignee:
              Unassigned
              Reporter:
              Jim Kellerman
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development