You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@s2graph.apache.org by "DOYUNG YOON (JIRA)" <ji...@apache.org> on 2016/03/09 01:59:41 UTC

[jira] [Created] (S2GRAPH-58) Offset query option is not working with tall row schema

DOYUNG YOON created S2GRAPH-58:
----------------------------------

             Summary: Offset query option is not working with tall row schema
                 Key: S2GRAPH-58
                 URL: https://issues.apache.org/jira/browse/S2GRAPH-58
             Project: S2Graph
          Issue Type: Bug
            Reporter: DOYUNG YOON
            Assignee: DOYUNG YOON


S2GRAPH-50 implemented new schema(tall row) for HBase Storage, but there were few missing parts our test cases missed out. 

One thing I found out is `offset` option on queryParam is not working. in wide row schema, we used `ColumnPaginationFilter` which accept offset and limit as parameter. in tall row schema, we need to change to use `PageFilter` and unlike `ColumnPaginationFilter`, `PageFilter` does not accept offset as parameter. 

I think we can either throw exception by checking compatibility on label's schema version and query, or we can implicitly fetch `offset + limit` key values and take only from offset index to limit. 

Or we can actually provide different option like providing cursors and let user to parse each cursors to paginate next chunk of edges. I am open to any suggestion, so feel free to give any opinion on this.




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