Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1115

Improve GraphTraversal and GraphTraversalSource Reference Docs

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.1.1-incubating
    • 3.1.5
    • documentation
    • None

    Description

      We have the lambda representation of these steps, but not the traversal representation such as map(traversal), flatMap(traversal), filter(traversal), etc.. It would be futher good to cover the various with... methods like withPath(), withSideEffect(), etc.

      Attachments

        Issue Links

          Activity

            dkuppitz You feel like handling this one?

            okram Marko A. Rodriguez added a comment - dkuppitz You feel like handling this one?

            Expanded the scope of this issue a little bit to cover the GraphTraversalSource.with...() methods.

            spmallette Stephen Mallette added a comment - Expanded the scope of this issue a little bit to cover the GraphTraversalSource.with...() methods.
            githubbot ASF GitHub Bot added a comment -

            GitHub user dkuppitz opened a pull request:

            https://github.com/apache/tinkerpop/pull/431

            TINKERPOP-1115 Improve GraphTraversal and GraphTraversalSource Reference Docs

            https://issues.apache.org/jira/browse/TINKERPOP-1115

            Let's get this merged. I'll keep the ticket open and add the `GraphTraversalSource` stuff later.

            VOTE: +1

            You can merge this pull request into a Git repository by running:

            $ git pull https://github.com/apache/tinkerpop TINKERPOP-1115

            Alternatively you can review and apply these changes as the patch at:

            https://github.com/apache/tinkerpop/pull/431.patch

            To close this pull request, make a commit to your master/trunk branch
            with (at least) the following in the commit message:

            This closes #431


            commit 54c08a7ff68e08f8c1f8b9f850086260e39d1b2d
            Author: Daniel Kuppitz <daniel_kuppitz@hotmail.com>
            Date: 2016-09-16T14:26:19Z

            Added documentation for the traversal representations of general steps (map, flatMap, filter, sideEffect, branch).


            githubbot ASF GitHub Bot added a comment - GitHub user dkuppitz opened a pull request: https://github.com/apache/tinkerpop/pull/431 TINKERPOP-1115 Improve GraphTraversal and GraphTraversalSource Reference Docs https://issues.apache.org/jira/browse/TINKERPOP-1115 Let's get this merged. I'll keep the ticket open and add the `GraphTraversalSource` stuff later. VOTE: +1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/tinkerpop TINKERPOP-1115 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tinkerpop/pull/431.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #431 commit 54c08a7ff68e08f8c1f8b9f850086260e39d1b2d Author: Daniel Kuppitz <daniel_kuppitz@hotmail.com> Date: 2016-09-16T14:26:19Z Added documentation for the traversal representations of general steps (map, flatMap, filter, sideEffect, branch).
            githubbot ASF GitHub Bot added a comment -

            Github user twilmes commented on the issue:

            https://github.com/apache/tinkerpop/pull/431

            Looking good.

            VOTE: +1

            githubbot ASF GitHub Bot added a comment - Github user twilmes commented on the issue: https://github.com/apache/tinkerpop/pull/431 Looking good. VOTE: +1
            githubbot ASF GitHub Bot added a comment -

            Github user okram commented on the issue:

            https://github.com/apache/tinkerpop/pull/431

            VOTE +1. Thanks.

            githubbot ASF GitHub Bot added a comment - Github user okram commented on the issue: https://github.com/apache/tinkerpop/pull/431 VOTE +1. Thanks.
            githubbot ASF GitHub Bot added a comment -

            Github user asfgit closed the pull request at:

            https://github.com/apache/tinkerpop/pull/431

            githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tinkerpop/pull/431
            dkuppitz Daniel Kuppitz added a comment -

            GraphTraversal docs were improved via https://github.com/apache/tinkerpop/pull/431.

            GraphTraversalSource still need to be done.

            dkuppitz Daniel Kuppitz added a comment - GraphTraversal docs were improved via https://github.com/apache/tinkerpop/pull/431 . GraphTraversalSource still need to be done.

            This was completed. dkuppitz Can you please close this ticket with the appropriate "Fix Version."

            okram Marko A. Rodriguez added a comment - This was completed. dkuppitz Can you please close this ticket with the appropriate "Fix Version."

            I was going to say the same thing, but given his last comment, it seems there is more work to do. GraphTraversalSource still needs to be documented. Can that be done this week in time for release?

            spmallette Stephen Mallette added a comment - I was going to say the same thing, but given his last comment, it seems there is more work to do. GraphTraversalSource still needs to be documented. Can that be done this week in time for release?

            People

              dkuppitz Daniel Kuppitz
              okram Marko A. Rodriguez
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: