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

ASF GitHub Bot commented on TINKERPOP-2727:
-------------------------------------------

spmallette commented on pull request #1596:
URL: https://github.com/apache/tinkerpop/pull/1596#issuecomment-1083005199


   i think i've addressed the issues here. for documentation of `has()` i added 
some details to the reference docs but ultimately i think we need to grow the[ 
Gremlin Semantics 
section](https://github.com/apache/tinkerpop/blob/master/docs/src/dev/provider/gremlin-semantics.asciidoc)
 to cover these sorts of details.


-- 
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]


> HasContainer should allow a null key
> ------------------------------------
>
>                 Key: TINKERPOP-2727
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2727
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.5.2
>            Reporter: Florian Hockmann
>            Priority: Minor
>
> TINKERPOP-2605 changed {{null}} semantics to allow {{null}} as an argument in 
> more places so that {{has(null)}} should not throw a NPE any more and instead 
> filter all traversers out.
> {{HasContainer}} however still throws a NPE if the {{key}} is {{{}null{}}}.
> I discovered this when trying to update JanusGraph [as that folds in a 
> {{HasContainer}} like 
> this|https://github.com/JanusGraph/janusgraph/blob/4246d49cee46d549d752515f3956cd2d59f1fd0a/janusgraph-core/src/main/java/org/janusgraph/graphdb/tinkerpop/optimize/step/HasStepFolder.java#L266]
>  which leads to the NPE and thus failing tests.
> The same behavior can also be produced with just TinkerGraph and for example 
> the following traversals:
> {code:java}
> g.V().has(null, 1)
> g.V().has(null, P.neq(null)){code}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to