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

addE() should allow dynamic edge labels

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.1
    • Component/s: process
    • Labels:
      None

      Description

      We can create vertices with dynamic labels, e.g:

      ...addV().property(label, select("x"))...
      

      This approach doesn't work for edges as we don't have a parameterless addE overload. However, I think we can allow addE() and use Edge.DEFAULT_LABEL, just like we do for vertices; then the aforementioned approach would work.

      As an aside: I would prefer addV(traversal) and addE(traversal) over addX().property(label, ...).

        Attachments

          Activity

            People

            • Assignee:
              okram Marko A. Rodriguez
              Reporter:
              dkuppitz Daniel Kuppitz

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment