[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/882 ---

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207509387 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java --- @@ -2451,6 +2452,24 @@ else if

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207377923 --- Diff: docs/src/recipes/shortest-path.asciidoc --- @@ -136,3 +158,44 @@ g.withSack(0.0).V().as("from"). <1> <7> Order the output by the

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207374407 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/Traversal.java --- @@ -496,15 +496,17 @@ public default void

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207373358 --- Diff: docs/src/recipes/shortest-path.asciidoc --- @@ -48,6 +48,17 @@ course, it is possible for there to be more than one path in the graph of the

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207197353 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java --- @@ -2451,6 +2452,24 @@ else if

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207191097 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/Traversal.java --- @@ -496,15 +496,17 @@ public default void

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207183855 --- Diff: docs/src/recipes/shortest-path.asciidoc --- @@ -136,3 +158,44 @@ g.withSack(0.0).V().as("from"). <1> <7> Order the output by the

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r207183373 --- Diff: docs/src/recipes/shortest-path.asciidoc --- @@ -48,6 +48,17 @@ course, it is possible for there to be more than one path in the graph of the

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206210239 --- Diff: docs/src/reference/the-traversal.asciidoc --- @@ -2489,6 +2489,62 @@

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user dkuppitz commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206207720 --- Diff: docs/src/reference/the-traversal.asciidoc --- @@ -2489,6 +2489,62 @@

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206194431 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/remote/RemoteGraph.java --- @@ -60,6 +60,10 @@ test =

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206188786 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/search/path/ShortestPathVertexProgram.java --- @@ -0,0 +1,581 @@

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206182822 --- Diff: docs/src/reference/the-traversal.asciidoc --- @@ -2489,6 +2489,62 @@

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user spmallette commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r206111874 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/step/map/ShortestPathVertexProgramStep.java --- @@ -0,0

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

Github user twilmes commented on a diff in the pull request: https://github.com/apache/tinkerpop/pull/882#discussion_r201547477 --- Diff: gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/search/path/ShortestPathVertexProgram.java --- @@ -0,0 +1,557 @@

[GitHub] tinkerpop pull request #882: TINKERPOP-1990 Add a shortestPath() step

GitHub user dkuppitz opened a pull request: https://github.com/apache/tinkerpop/pull/882 TINKERPOP-1990 Add a shortestPath() step Implemented `ShortestPathVertexProgram` and `ShortestPathVertexProgramStep`. `docker/build.sh -t -i -n -d` passed. VOTE: +1 I