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

Redefine Identity Of HBase Configuration

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.90.2
    • Fix Version/s: 0.92.0
    • Component/s: Client, IPC/RPC
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Judging from the javadoc in HConnectionManager, sharing connections across multiple clients going to the same cluster is supposedly a good thing. However, the fact that there is a one-to-one mapping between a configuration and connection instance, kind of works against that goal. Specifically, when you create HTable instances using a given Configuration instance and a copy thereof, we end up with two distinct HConnection instances under the covers. Is this really expected behavior, especially given that the configuration instance gets cloned a lot?

      Here, I'd like to play devil's advocate and propose that we "deep-compare" HBaseConfiguration instances, so that multiple HBaseConfiguration instances that have the same properties map to the same HConnection instance. In case one is "concerned that a single HConnection is insufficient for sharing amongst clients", to quote the javadoc, then one should be able to mark a given HBaseConfiguration instance as being "uniquely identifiable".

      Note that "sharing connections makes clean up of HConnection instances a little awkward", unless of course, you apply the change described in HBASE-3766.

        Attachments

        1. HBASE-3777.patch
          16 kB
          Karthick Sankarachary
        2. HBASE-3777-V2.patch
          26 kB
          Karthick Sankarachary
        3. HBASE-3777-V3.patch
          38 kB
          Karthick Sankarachary
        4. HBASE-3777-V4.patch
          40 kB
          Karthick Sankarachary
        5. HBASE-3777-V6.patch
          54 kB
          Karthick Sankarachary
        6. 3777-TOF.patch
          0.9 kB
          Ted Yu
        7. HBASE-3777-V8.0.90.4.backport.patch
          76 kB
          Bright Fulton

          Issue Links

            Activity

              People

              • Assignee:
                karthick Karthick Sankarachary
                Reporter:
                karthick Karthick Sankarachary
              • Votes:
                1 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: