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/02/04 10:37:40 UTC

[jira] [Created] (S2GRAPH-43) Bug on offset when Interval is given on query.

DOYUNG YOON created S2GRAPH-43:
----------------------------------

             Summary: Bug on offset when Interval is given on query.
                 Key: S2GRAPH-43
                 URL: https://issues.apache.org/jira/browse/S2GRAPH-43
             Project: S2Graph
          Issue Type: Bug
            Reporter: DOYUNG YOON
            Assignee: DOYUNG YOON
            Priority: Trivial


Since {{degreeEdge}} that store how many edges are exist from this source vertex use empty byte array as qualifier, it always comes at first. 

Above is reason why we add 1 on limit when offset is 0(if offset is 0, then we need to return degree too so storage level limit should be increased by 1).

This is not actually true when "interval" is given.
When query has interval option, then it use columnRangeFilter, and with offset 0 and interval option, then current implementation return 1 more result. 

I think adding 1 on provided limit should check if interval is specified on queryParam.




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