Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.99.0, hbase-10070
    • Component/s: None
    • Labels:
      None

      Description

      Currently the way cache works is that the meta row is stored together for all replicas of a region, so if some replicas are in recovery, getting locations for a region will still go to cache only and return null locations for these. Multi-get currently ignores such replicas. It should instead try to get location again from meta if any replica is null.

      1. HBASE-10794.patch
        13 kB
        Sergey Shelukhin
      2. HBASE-10794.patch
        33 kB
        Sergey Shelukhin
      3. HBASE-10794.03.patch
        36 kB
        Sergey Shelukhin
      4. HBASE-10794.02.patch
        34 kB
        Sergey Shelukhin
      5. HBASE-10794.02.addendum.patch
        3 kB
        Sergey Shelukhin
      6. HBASE-10794.01.patch
        33 kB
        Sergey Shelukhin
      7. 0033-HBASE-10794-multi-get-should-handle-replica-location.patch
        37 kB
        Enis Soztutar

        Issue Links

          Activity

          Hide
          Enis Soztutar added a comment -

          Closing this issue after 0.99.0 release.

          Show
          Enis Soztutar added a comment - Closing this issue after 0.99.0 release.
          Hide
          Hudson added a comment -

          FAILURE: Integrated in HBase-TRUNK #5245 (See https://builds.apache.org/job/HBase-TRUNK/5245/)
          HBASE-10794 multi-get should handle replica location missing from cache (enis: rev 579f305bd0fb5673a82d3eeb74660db7a0d6ddd8)

          • hbase-server/src/main/java/org/apache/hadoop/hbase/client/CoprocessorHConnection.java
          • hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java
          • hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java
          • hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClusterConnection.java
          • hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionAdapter.java
          • hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
          • hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java
          Show
          Hudson added a comment - FAILURE: Integrated in HBase-TRUNK #5245 (See https://builds.apache.org/job/HBase-TRUNK/5245/ ) HBASE-10794 multi-get should handle replica location missing from cache (enis: rev 579f305bd0fb5673a82d3eeb74660db7a0d6ddd8) hbase-server/src/main/java/org/apache/hadoop/hbase/client/CoprocessorHConnection.java hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClusterConnection.java hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionAdapter.java hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java
          Hide
          Enis Soztutar added a comment -

          Committed to master as part of hbase-10070 branch merge

          Show
          Enis Soztutar added a comment - Committed to master as part of hbase-10070 branch merge
          Hide
          Enis Soztutar added a comment -

          Attaching rebased patch for master that is committed

          Show
          Enis Soztutar added a comment - Attaching rebased patch for master that is committed
          Hide
          Sergey Shelukhin added a comment -

          committed to branch

          Show
          Sergey Shelukhin added a comment - committed to branch
          Hide
          Sergey Shelukhin added a comment -

          Rebased patch, merged in the addendum. No substantial changes so I will commit to branch later today

          Show
          Sergey Shelukhin added a comment - Rebased patch, merged in the addendum. No substantial changes so I will commit to branch later today
          Hide
          Sergey Shelukhin added a comment -

          addendum to the 2nd patch to fix a bug; when upstream patches are committed I'll figure out how to combine all this stuff, it will be committed together

          Show
          Sergey Shelukhin added a comment - addendum to the 2nd patch to fix a bug; when upstream patches are committed I'll figure out how to combine all this stuff, it will be committed together
          Hide
          Sergey Shelukhin added a comment -

          Another iteration, on top of two patches from blocking jiras

          Show
          Sergey Shelukhin added a comment - Another iteration, on top of two patches from blocking jiras
          Hide
          Sergey Shelukhin added a comment -

          include other changes that are not part of HBASE-10634

          Show
          Sergey Shelukhin added a comment - include other changes that are not part of HBASE-10634
          Hide
          Sergey Shelukhin added a comment -

          This patch is on top of two blocking patches

          Show
          Sergey Shelukhin added a comment - This patch is on top of two blocking patches
          Hide
          Sergey Shelukhin added a comment -
          Show
          Sergey Shelukhin added a comment - Enis Soztutar Devaraj Das fyi

            People

            • Assignee:
              Sergey Shelukhin
              Reporter:
              Sergey Shelukhin
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development