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

Avoid passing unexpected cell to ScanQueryMatcher when optimize SEEK to SKIP

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0, 2.0.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      ScanQueryMatcher.MatchCode qcode = matcher.match(cell);
      qcode = optimize(qcode, cell);
      

      The optimize method may change the MatchCode from SEEK_NEXT_COL/SEEK_NEXT_ROW to SKIP. But it still pass the next cell to ScanQueryMatcher. It will get wrong result when use some filter, etc. ColumnCountGetFilter. It just count the columns's number. If pass a same column to this filter, the count result will be wrong. So we should avoid passing cell to ScanQueryMatcher when optimize SEEK to SKIP.

        Attachments

        1. 0001-add-one-ut-testWithColumnCountGetFilter.patch
          6 kB
          Guanghao Zhang
        2. 17958-add.txt
          2 kB
          Lars Hofhansl
        3. HBASE-17958-branch-1.patch
          33 kB
          Guanghao Zhang
        4. HBASE-17958-branch-1.patch
          33 kB
          Duo Zhang
        5. HBASE-17958-branch-1.patch
          33 kB
          Guanghao Zhang
        6. HBASE-17958-branch-1.patch
          33 kB
          Guanghao Zhang
        7. HBASE-17958-v1.patch
          9 kB
          Guanghao Zhang
        8. HBASE-17958-v2.patch
          14 kB
          Guanghao Zhang
        9. HBASE-17958-v3.patch
          16 kB
          Guanghao Zhang
        10. HBASE-17958-v4.patch
          22 kB
          Guanghao Zhang
        11. HBASE-17958-v5.patch
          23 kB
          Guanghao Zhang
        12. HBASE-17958-v6.patch
          23 kB
          Guanghao Zhang
        13. HBASE-17958-v7.patch
          23 kB
          Guanghao Zhang
        14. HBASE-17958-v7.patch
          23 kB
          Guanghao Zhang

          Issue Links

            Activity

              People

              • Assignee:
                zghaobac Guanghao Zhang
                Reporter:
                zghaobac Guanghao Zhang
              • Votes:
                0 Vote for this issue
                Watchers:
                10 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: