[jira] [Commented] (TINKERPOP-2099) Property setting with null has different behavior between add and update

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TINKERPOP-2099?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16861338#comment-16861338 ] 

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
 
 
   
 
----------------------------------------------------------------
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:
users@infra.apache.org


> 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
>            Assignee: 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)