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

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

Github user twilmes commented on the issue:

    https://github.com/apache/tinkerpop/pull/587
  
    This is an impressive set of performance improvements. I was testing and 
ran into an issue with this following query.  It ran successfully against 
`tp32`.
    
    ```
    graph = TinkerGraph.open()
    g = graph.traversal()
    
g.addV().as("first").repeat(addE("next").to(addV()).inV()).times(5).addE("next").to(select("first"))
    
    java.lang.IllegalStateException: The traversal strategies are complete and 
the traversal can no longer be modulated
        at 
org.apache.tinkerpop.gremlin.process.traversal.Traversal$Exceptions.traversalIsLocked(Traversal.java:274)
        at 
org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.applyStrategies(DefaultTraversal.java:112)
        at 
org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.applyStrategies(DefaultTraversal.java:129)
        at 
org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.hasNext(DefaultTraversal.java:186)
        at 
org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
        at 
org.apache.tinkerpop.gremlin.console.Console$_closure3.doCall(Console.groovy:234)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at 
org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
        at 
org.codehaus.groovy.runtime.metaclass.ClosureMetaClass.invokeMethod(ClosureMetaClass.java:294)
 ...
    ```


> Improve performance of mutating traversals
> ------------------------------------------
>
>                 Key: TINKERPOP-1642
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1642
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.2.4
>            Reporter: stephen mallette
>            Assignee: stephen mallette
>            Priority: Critical
>
> Make an attempt improve performance of mutating traversals. Some general 
> goals for this ticket should be to:
> 1. Improve performance parity of structure and process methods for modifying 
> the graph - structure is pretty far ahead.
> 2. Improve the speed of large chained mutating traversals - see 
> http://stackoverflow.com/q/41926409/1831717
> 3. Determine ways to make these performance improvements in {{gremlin-core}} 
> so that they may benefit all {{Graph}} implementations
> 4. Improve the speed of TinkerGraph specifically if possible as it is used 
> for subgraphing functions in many cases and ends up being useful regardless 
> of the graph implementation ultimately used.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to