hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Duo Zhang (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-17595) Add partial result support for small/limited scan
Date Tue, 14 Feb 2017 07:49:41 GMT

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

Duo Zhang commented on HBASE-17595:

After reviewing the scanner implementation at RS side, I found that we can make sure if there
are more cells in row in the current implementation. In RegionScannerImpl, we will use nextKv
to determine if there is a row change. And the kv returned has already been filtered with
Filter.filterKeyValue(in StoreScanner). So we can just rename 'mayHaveMoreCellsInRow' to 'hasMoreCellsInRow'.
This will grealy simplify the implementation at client side.

> Add partial result support for small/limited scan
> -------------------------------------------------
>                 Key: HBASE-17595
>                 URL: https://issues.apache.org/jira/browse/HBASE-17595
>             Project: HBase
>          Issue Type: Sub-task
>          Components: asyncclient, Client, scan
>    Affects Versions: 2.0.0, 1.4.0
>            Reporter: Duo Zhang
>             Fix For: 2.0.0, 1.4.0
> The partial result support is marked as a 'TODO' when implementing HBASE-17045. And when
implementing HBASE-17508, we found that if we make small scan share the same logic with general
scan, the scan request other than open scanner will not have the small flag so the server
may return  partial result to the client and cause some strange behavior. It is solved by
modifying the logic at server side, but this means the 1.4.x client is not safe to contact
with earlier 1.x server. So we'd better address the problem at client side. Marked as blocker
as this issue should be finished before any 2.x and 1.4.x releases.

This message was sent by Atlassian JIRA

View raw message