hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-15247) InclusiveStopFilter does not respect reverse Filter property
Date Wed, 16 Mar 2016 18:09:33 GMT

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

Hudson commented on HBASE-15247:
--------------------------------

FAILURE: Integrated in HBase-0.98-on-Hadoop-1.1 #1186 (See [https://builds.apache.org/job/HBase-0.98-on-Hadoop-1.1/1186/])
HBASE-15247 InclusiveStopFilter does not respect reverse Filter (apurtell: rev 92eea6db5cbe0abfd4acf5575929863bac2d8c02)
* hbase-server/src/test/java/org/apache/hadoop/hbase/filter/TestFilter.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/filter/InclusiveStopFilter.java


> InclusiveStopFilter does not respect reverse Filter property
> ------------------------------------------------------------
>
>                 Key: HBASE-15247
>                 URL: https://issues.apache.org/jira/browse/HBASE-15247
>             Project: HBase
>          Issue Type: Bug
>          Components: Filters
>    Affects Versions: 2.0.0, 1.1.1
>            Reporter: Rick Moritz
>            Assignee: Amal Joshy
>             Fix For: 2.0.0, 1.3.0, 0.98.18
>
>         Attachments: HBASE-15247-branch-1.1.patch, HBASE-15247.patch
>
>
> InclusiveStopFilter only works with non-reversed Scans, it will not filter for reversed
Scans, because it doesn't flip the cmp-operand in the reversed case. In fact, it doesn't even
use the Filter.reverse flag.
> it should be something like this:
> if (reversed) {
>             if (cmp > 0) {
>                 done = true;
>             }
>         }
>         else {
>             if (cmp < 0) {
>                 done = true;
>             }
>         }



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

Mime
View raw message