You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2016/03/16 12:35:33 UTC
[jira] [Commented] (HBASE-15247) InclusiveStopFilter does not
respect reverse Filter property
[ https://issues.apache.org/jira/browse/HBASE-15247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15197222#comment-15197222 ]
Hudson commented on HBASE-15247:
--------------------------------
ABORTED: Integrated in HBase-0.98-matrix #312 (See [https://builds.apache.org/job/HBase-0.98-matrix/312/])
HBASE-15247 InclusiveStopFilter does not respect reverse Filter (apurtell: rev 92eea6db5cbe0abfd4acf5575929863bac2d8c02)
* hbase-client/src/main/java/org/apache/hadoop/hbase/filter/InclusiveStopFilter.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/filter/TestFilter.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)