HBase
  1. HBase
  2. HBASE-10578

For the same row key, the KV in the newest StoreFile should be returned

    Details

    • Type: Bug Bug
    • Status: Patch Available
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 0.89-fb, 0.99.0, 0.98.2
    • Fix Version/s: 0.89-fb
    • Component/s: Scanners
    • Labels:
      None

      Description

      When multiple scanners have the same KV, HBase should pick the "newest" one.
      i.e. pick the KV from the store file with the largest seq id.

      In the KeyValueHeap generalizedSeek implementation, we seem to prefer the "current"
      scanner over the scanners in the heap – THIS IS WRONG.

      The diff adds a unit test to make sure that bulk loads correctly. And fixes the issue.

      1. HBASE-10578-v2.patch
        10 kB
        Amitanand Aiyer
      2. HBASE-10578.patch
        9 kB
        Amitanand Aiyer

        Activity

        Hide
        Liyin Tang added a comment -

        Nice finding !

        Show
        Liyin Tang added a comment - Nice finding !
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12630159/HBASE-10578.patch
        against trunk revision .
        ATTACHMENT ID: 12630159

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 4 new or modified tests.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/8759//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12630159/HBASE-10578.patch against trunk revision . ATTACHMENT ID: 12630159 +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/8759//console This message is automatically generated.
        Hide
        Amitanand Aiyer added a comment -

        This file is intended to be applied over 0.89-fb.

        Not sure why HadoopQA tries to apply this over opensource trunk, and complain about it.

        Show
        Amitanand Aiyer added a comment - This file is intended to be applied over 0.89-fb. Not sure why HadoopQA tries to apply this over opensource trunk, and complain about it.
        Hide
        stack added a comment -

        Amitanand Aiyer Yeah, our little hadoopqa bot just runs whatever is posted against trunk... just ignore.

        Show
        stack added a comment - Amitanand Aiyer Yeah, our little hadoopqa bot just runs whatever is posted against trunk... just ignore.
        Hide
        stack added a comment -

        We have this issue in trunk too. The patch looks like it'll apply too...

        Show
        stack added a comment - We have this issue in trunk too. The patch looks like it'll apply too...
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12630192/HBASE-10578-v2.patch
        against trunk revision .
        ATTACHMENT ID: 12630192

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 4 new or modified tests.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/8761//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12630192/HBASE-10578-v2.patch against trunk revision . ATTACHMENT ID: 12630192 +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 4 new or modified tests. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/8761//console This message is automatically generated.

          People

          • Assignee:
            Amitanand Aiyer
            Reporter:
            Amitanand Aiyer
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:

              Development