Cole-Greer commented on code in PR #3483: URL: https://github.com/apache/tinkerpop/pull/3483#discussion_r3540409476
########## docs/src/dev/provider/gremlin-semantics.asciidoc: ########## @@ -587,6 +587,69 @@ See: link:https://github.com/apache/tinkerpop/tree/x.y.z/gremlin-core/src/main/j link:https://github.com/apache/tinkerpop/tree/x.y.z/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/AddVertexStartStep.java[source (start)], link:https://tinkerpop.apache.org/docs/x.y.z/reference/#addvertex-step[reference] +=== addLabel() + +*Description:* Adds one or more labels to an element. + +*Syntax:* `addLabel(String label, String... moreLabels)` | `addLabel(Traversal<?, String> labelTraversal)` + +[width="100%",options="header"] +|========================================================= +|Start Step |Mid Step |Modulated |Domain |Range +|N |Y |N |`Element` |`Element` +|========================================================= + +*Arguments:* + +* `label` - The label to add. +* `moreLabels` - Additional labels to add. +* `labelTraversal` - A traversal that produces labels to add. + +*Considerations:* Review Comment: I've reworded this for clarity, but yes singlelabel does imply immutability, because there is no atomic operation to replace labels, and there labels cannot be added or removed without violating the cardinality. This also keeps `LabelCardinality.ONE` consistent with the legacy TP3 semantics which I think is nice. -- 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]
