[ 
https://issues.apache.org/jira/browse/S2GRAPH-63?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

DOYUNG YOON updated S2GRAPH-63:
-------------------------------
    Fix Version/s: 0.1.0

> 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
>            Assignee: DOYUNG YOON
>              Labels: newbie, write
>             Fix For: 0.1.0
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> 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)

Reply via email to