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

Robert Dale commented on TINKERPOP-1468:
----------------------------------------

Then again, you do have the stacktrace pointing right to the optional() method. 
I think the next logical step is to check the API. The API clearly shows that 
it takes a Traversal parameter. I hope at that point it's obvious what needs to 
be done. It may not be user friendly in the sense of hand-holding, but it's a 
lot better than most SQL error messages.  So, yes, I would close as "won't fix".

> GraphTraversal.optional throws a NPE if no traversal is provided
> ----------------------------------------------------------------
>
>                 Key: TINKERPOP-1468
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1468
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.2.2
>            Reporter: Ted Wilmes
>              Labels: trivial
>
> {{GraphTraversal.optional}} throws a NPE if no traversal is provided.  We 
> should probably throw a friendlier exception with instructions to provide a 
> traversal as an argument.
> {code}
> gremlin> g.V().optional()
> java.lang.NullPointerException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.lang.NullPointerException
>       at 
> org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal.optional(GraphTraversal.java:1316)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to