[
https://issues.apache.org/jira/browse/TINKERPOP-3137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17928568#comment-17928568
]
ASF GitHub Bot commented on TINKERPOP-3137:
-------------------------------------------
spmallette opened a new pull request, #3055:
URL: https://github.com/apache/tinkerpop/pull/3055
https://issues.apache.org/jira/browse/TINKERPOP-3137
Removed validation for property values, but left it for enums like
`Direction`, `T` and `Merge` where it wouldn't make sense.
VOTE +1
> Allow null to be used as Map value to mergeV and mergeE
> -------------------------------------------------------
>
> Key: TINKERPOP-3137
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3137
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.7.3
> Reporter: Stephen Mallette
> Assignee: Stephen Mallette
> Priority: Minor
>
> Not sure why it was explicitly blocked but there is validation that prevents
> use of {{null}} as a property value for {{Map}} given to {{mergeV/E}}. That
> validation should be removed because some graphs allow the storage of
> {{null}} and the general Gremlin semantics for setting a property to {{null}}
> for graphs that don't support storage is to remove the property. Removing the
> validation will let {{mergeV/E}} adhere to those semantics.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
