[
https://issues.apache.org/jira/browse/TINKERPOP-1793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16183115#comment-16183115
]
ASF GitHub Bot commented on TINKERPOP-1793:
-------------------------------------------
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/724
Manual tests (see
[SO](https://stackoverflow.com/questions/46363737/tinkerpop-gremlin-merge-vertices-and-edges))
worked like a charm.
VOTE: +1
> addE() should allow dynamic edge labels
> ---------------------------------------
>
> Key: TINKERPOP-1793
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1793
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Reporter: Daniel Kuppitz
> Assignee: Marko A. Rodriguez
>
> We can create vertices with dynamic labels, e.g:
> {noformat}
> ...addV().property(label, select("x"))...
> {noformat}
> 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, ...)}}.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
