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/04/18 10:37:25 UTC

[jira] [Created] (S2GRAPH-63) Condition on partition strong edges and weak edges on mutateEdges is wrong.

DOYUNG YOON created S2GRAPH-63:
----------------------------------

             Summary: Condition on partition strong edges and weak edges on mutateEdges is wrong.
                 Key: S2GRAPH-63
                 URL: https://issues.apache.org/jira/browse/S2GRAPH-63
             Project: S2Graph
          Issue Type: Bug
            Reporter: DOYUNG YOON


Storage#mutateEdges use following condition to separate strong edges and weak edges.

{noformat}
 val (strongEdges, weakEdges) =
      (edges.partition(e => e.label.consistencyLevel == "strong" || e.op == GraphUtil.operations("insertBulk")))
{noformat}

I think current condition is wrong. the condition should be following.
{noformat}
 val (strongEdges, weakEdges) =
      (edges.partition(e => e.label.consistencyLevel == "strong" && e.op != GraphUtil.operations("insertBulk")
{noformat}

Because of above bug, insertBulk is not working as intended(not lookup snapshotEdge and just build and fire RPCs).



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