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

Use setters instead of return values for handing back statistics from HRegion methods

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.98.10
    • 1.0.0, 1.1.0, 0.98.11, 0.98.10.1
    • None

    Description

      In HBASE-5162 (and backports such as HBASE-12729) we modified some HRegion methods to return statistics for consumption by callers. The statistics are ultimately passed back to the client as load feedback.

      lhofhansl thinks handing back this information as return values from HRegion methods is a weird mix of concerns. This also produced a difficult to anticipate binary compatibility issue with Phoenix. There was no compile time issue because the code of course was not structured to assign from a method returning void, yet the method signature changes so the JVM cannot resolve it if older Phoenix binaries are installed into a 0.98.10 release. Let's change the HRegion methods back to returning 'void' and use setters instead. Officially we don't support use of HRegion (HBASE-12566) but we do not need to go out of our way to break things (smile) so I would also like to make a patch release containing just this change to help out our sister project.

      Attachments

        1. hbase-12979-v0-0.98.patch
          3 kB
          Jesse Yates
        2. hbase-12979-v0-master.patch
          3 kB
          Jesse Yates

        Activity

          People

            jesse_yates Jesse Yates
            apurtell Andrew Kyle Purtell
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: