Cole-Greer commented on PR #3252: URL: https://github.com/apache/tinkerpop/pull/3252#issuecomment-3459135923
> I don't understand the need for this PR still after reviewing it. Can you maybe expand a bit about it in the documentation? What problem exactly is it trying to fix? This PR is really a followup to an unintended consequence stemming from https://github.com/apache/tinkerpop/pull/3133. The issue with directly allowing ID's to be passed into from/to, it creates a weird asymmetry for graphs which use Strings as their ids. For example `g.addE().from(1).to(2)` would work fine and create an edge from `V[1]` to `V[2]`, however if the graph used String ids, the equivalent traversal (`g.addE().from("a").to("b")`) does not work, as the Strings are interpreted as path labels (converted to `__.select("a")`) instead of vertex ids. There are some ugly workarounds which can partially mitigate this issue, but overall there is no generalized fix which allows all traversals to behave the same regardless of the vertex id type. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
