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

addE() should allow dynamic edge labels

    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

          Issue Links

            Activity

              People

              • Assignee:
                okram Marko A. Rodriguez
                Reporter:
                dkuppitz Daniel Kuppitz
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: