You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Ankit Singhal (JIRA)" <ji...@apache.org> on 2018/01/09 07:51:00 UTC
[jira] [Resolved] (PHOENIX-4467) Scans with filter returning all
the versions
[ https://issues.apache.org/jira/browse/PHOENIX-4467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ankit Singhal resolved PHOENIX-4467.
------------------------------------
Resolution: Fixed
It is fixed as part of HBASE-19696
> Scans with filter returning all the versions
> --------------------------------------------
>
> Key: PHOENIX-4467
> URL: https://issues.apache.org/jira/browse/PHOENIX-4467
> Project: Phoenix
> Issue Type: Sub-task
> Reporter: Ankit Singhal
> Assignee: Ankit Singhal
> Labels: HBase-2.0
> Fix For: 5.0.0
>
> Attachments: PHOENIX-4467_workaround.patch
>
>
> Even if we set max versions in the scan to 1, HBase is still returning all the versions if there is a filter.
> It seems to be a side-effect of HBASE-17125. where if INCLUDE_AND_NEXT_COL is the return code from the filter, we are not checking the version of the cell even though the documentation says.
> {code}
> @InterfaceAudience.Public
> public enum ReturnCode {
> /**
> * Include the Cell
> */
> INCLUDE,
> /**
> * Include the Cell and seek to the next column skipping older versions.
> */
> INCLUDE_AND_NEXT_COL,
> }
> {code}
> In UserScanQueryMatcher#mergeFilterResponse()
> {code}
> case INCLUDE_AND_NEXT_COL:
> if (matchCode == MatchCode.INCLUDE) {
> matchCode = MatchCode.INCLUDE_AND_SEEK_NEXT_COL;
> }
> break;
> if (matchCode == MatchCode.INCLUDE_AND_SEEK_NEXT_COL
> || matchCode == MatchCode.INCLUDE_AND_SEEK_NEXT_ROW) {
> return matchCode;
> }
> // Now we will check versions again.
> if (curColCell == null || !CellUtil.matchingRowColumn(cell, curColCell)) {
> count = 0;
> curColCell = cell;
> }
> count += 1;
> return count > versionsAfterFilter ? MatchCode.SEEK_NEXT_COL : MatchCode.INCLUDE;
> {code}
> [~anoopsamjohn], It could be a bug in HBase
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)