[ 
https://issues.apache.org/jira/browse/TINKERPOP-1529?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15668059#comment-15668059
 ] 

ASF GitHub Bot commented on TINKERPOP-1529:
-------------------------------------------

Github user okram commented on the issue:

    https://github.com/apache/tinkerpop/pull/485
  
    ```
    gremlin> g.V().barrier().store('a').sideEffect(select('a')).explain()
    ==>Traversal Explanation
    
==================================================================================================================================================================
    Original Traversal                 [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a)])]
    
    ConnectiveStrategy           [D]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a)])]
    NoBarrierStrategy            [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a)])]
    RepeatUnrollStrategy         [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a)])]
    MatchPredicateStrategy       [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a)])]
    PathRetractionStrategy       [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    InlineFilterStrategy         [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    IncidentToAdjacentStrategy   [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    AdjacentToIncidentStrategy   [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    FilterRankingStrategy        [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    RangeByIsCountStrategy       [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    LazyBarrierStrategy          [O]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    TinkerGraphCountStrategy     [P]   [GraphStep(vertex,[]), NoOpBarrierStep, 
StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    TinkerGraphStepStrategy      [P]   [TinkerGraphStep(vertex,[]), 
NoOpBarrierStep, StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    ProfileStrategy              [F]   [TinkerGraphStep(vertex,[]), 
NoOpBarrierStep, StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    StandardVerificationStrategy [V]   [TinkerGraphStep(vertex,[]), 
NoOpBarrierStep, StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    
    Final Traversal                    [TinkerGraphStep(vertex,[]), 
NoOpBarrierStep, StoreStep(a), TraversalSideEffectStep([SelectOneStep(a), 
NoOpBarrierStep(2500)])]
    gremlin>
    ```
    `NoBarrierStep` doesn't seem to do anything.... ?


> LazyBarrierStrategy is too agressive
> ------------------------------------
>
>                 Key: TINKERPOP-1529
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1529
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.2.3
>            Reporter: Daniel Kuppitz
>            Assignee: Daniel Kuppitz
>
> There are scenarios where {{LazyBarrierStrategy}} changes the semantics of a 
> traversal:
> {noformat}
> gremlin> g = TinkerFactory.createModern().traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().store("a").out().select("a")
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> {noformat}
> This is actually not the result of {{store()}}, this is {{aggregate()}}. The 
> expected result for {{store()}} would be:
> {noformat}
> ==>[v[1]]
> ==>[v[1]]
> ==>[v[1]]
> ==>[v[1],v[2],v[3],v[4]]
> ==>[v[1],v[2],v[3],v[4]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> {noformat}
> Another issue, which should probably go into another ticket, is this:
> {noformat}
> gremlin> 
> g.withoutStrategies(LazyBarrierStrategy).V().store("a").out().select("a")
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> {noformat}
> That's it, the console is hanging at this point. Looks like 
> {{PathRetractionStrategy}} is the remaining troublemaker. With both 
> strategies excluded, we get the expected result:
> {noformat}
> gremlin> g.withoutStrategies(LazyBarrierStrategy, 
> PathRetractionStrategy).V().store("a").out().select("a")
> ==>[v[1]]
> ==>[v[1]]
> ==>[v[1]]
> ==>[v[1],v[2],v[3],v[4]]
> ==>[v[1],v[2],v[3],v[4]]
> ==>[v[1],v[2],v[3],v[4],v[5],v[6]]
> {noformat}



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

Reply via email to