Github user mpollmeier commented on the issue:

    https://github.com/apache/tinkerpop/pull/715
  
    That's likely because of the RepeatUnrollStrategy, which kicks in when 
there's a foreseeable number of iterations. Needs to be changed as well I 
guess. 
    
    -----Original Message-----
    From: Daniel Kuppitz <notificati...@github.com>
    To: apache/tinkerpop <tinker...@noreply.github.com>
    Cc: Michael Pollmeier <mich...@michaelpollmeier.com>, Author 
<aut...@noreply.github.com>
    Sent: Sat, 16 Sep 2017 1:53
    Subject: Re: [apache/tinkerpop] change behaviour of repeat step to be depth 
first search (DFS) (#715)
    
    Using the modern graph:
    
    ```
    gremlin> g.V().emit().repeat(both()).times(3).limit(15).path()
    ==>[v[1]]
    ==>[v[1],v[3]]
    ==>[v[1],v[2]]
    ==>[v[1],v[4]]
    ==>[v[1],v[3],v[1]]
    ==>[v[1],v[3],v[1],v[3]]
    ==>[v[1],v[3],v[1],v[2]]
    ==>[v[1],v[3],v[1],v[4]]
    ==>[v[1],v[3],v[4]]
    ==>[v[1],v[3],v[4],v[5]]
    ==>[v[1],v[3],v[4],v[3]]
    ==>[v[1],v[3],v[4],v[1]]
    ==>[v[1],v[3],v[6]]
    ==>[v[1],v[3],v[6],v[3]]
    ==>[v[1],v[2],v[1]]
    ```
    
    ^ This doesn't look like DFS to me. Row 3 and 4 should come much later.
    
    -- 
    You are receiving this because you authored the thread.
    Reply to this email directly or view it on GitHub:
    https://github.com/apache/tinkerpop/pull/715#issuecomment-329789246



---

Reply via email to