[
https://issues.apache.org/jira/browse/TINKERPOP-2099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16853149#comment-16853149
]
ASF GitHub Bot commented on TINKERPOP-2099:
-------------------------------------------
spmallette commented on pull request #1123: TINKERPOP-2099 Consistent behavior
of property() in relation to null values
URL: https://github.com/apache/tinkerpop/pull/1123
https://issues.apache.org/jira/browse/TINKERPOP-2099
See the upgrade documentation in the PR itself for more details on the fix.
Note that this is a breaking change as the behavior for the well used `addV()`
has changed when `null` is a property value.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> Property setting with null has different behavior between add and update
> ------------------------------------------------------------------------
>
> Key: TINKERPOP-2099
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2099
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.3.4
> Reporter: stephen mallette
> Priority: Minor
> Labels: breaking
> Fix For: 3.5.0
>
>
> Demonstrated via:
> {code}
> gremlin> graph = TinkerFactory.createModern()
> ==>tinkergraph[vertices:6 edges:6]
> gremlin> g = graph.traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V(1).property("x", 1).property("y", null).property("z", 2)
> The AddPropertyStep does not have a provided value: AddPropertyStep({key=[y]})
> Type ':help' or ':h' for help.
> Display stack trace? [yN]N
> gremlin> g.V(1).properties()
> ==>vp[name->marko]
> ==>vp[x->1]
> ==>vp[age->29]
> gremlin> g.addV("test").property("x", 1).property("y", null).property("z", 2)
> ==>v[13]
> gremlin> g.V(13).properties()
> ==>vp[x->1]
> ==>vp[z->2]
> {code}
> Make the {{AddVertexStep}} have similar functionality to
> {{AddPropertyStep}}...or the other way around.
> Added the breaking label here as either change will mean a change in behavior.
> Originally described here:
> https://groups.google.com/d/msg/gremlin-users/f1IHGb12eGY/i3UbaquXCAAJ
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
