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)

Reply via email to