hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-13491) Issue in FuzzyRowFilter#getNextForFuzzyRule
Date Wed, 22 Apr 2015 01:19:59 GMT

    [ https://issues.apache.org/jira/browse/HBASE-13491?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14506192#comment-14506192
] 

Andrew Purtell commented on HBASE-13491:
----------------------------------------

bq. Andy, the bug is not there in 98. I have checked at that time only and selected affects
versions. Sorry did not say explicitly.
The change to the filter picked back cleanly, looks relevant when inspecting the diff, and
the updated test passes. I didn't apply the test only first, though, so this could just be
a wash.

> Issue in FuzzyRowFilter#getNextForFuzzyRule
> -------------------------------------------
>
>                 Key: HBASE-13491
>                 URL: https://issues.apache.org/jira/browse/HBASE-13491
>             Project: HBase
>          Issue Type: Bug
>          Components: Filters
>    Affects Versions: 1.0.0
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>             Fix For: 2.0.0, 1.0.1, 1.1.0, 0.98.13, 1.2.0
>
>         Attachments: HBASE-13491-branch-1.1.patch, HBASE-13491-branch-1.1.patch, HBASE-13491.patch
>
>
> {code}
> for (int i = 0; i < result.length; i++) {
>       if (i >= fuzzyKeyMeta.length || fuzzyKeyMeta[i] == 1) {
>         result[i] = row[offset + i];
>         if (!order.isMax(row[i])) {
>           // this is "non-fixed" position and is not at max value, hence we can increase
it
>           toInc = i;
>         }
>       }
> {code}
> See we take row bytes with out considering the row offset.  The test cases are passing
as we pass 0 offset row bytes. Change in the test will reveal the bug.
> Came across this when I was working on HBASE-11425



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message