Github user spmallette commented on the issue:

    https://github.com/apache/tinkerpop/pull/818
  
    Thanks for looking into this issue. I saw your issue in JIRA and your SO 
question, but just haven't had a chance to really dig into it. I can see how 
your solution would get rid of the `NullPointerException`, but do you 
understand the root cause of why this happens in the first place? Could you 
perhaps add your reasoning as to why this is the right fix in that context in 
the description of this pull request?
    
    Also, if this is the right fix, could you get rid of the use of `stream()` 
in favor of TinkerPop's `IteratorUtils.filter()` - we've found `stream()` to 
sadly be far less performant. 
    
    Finally, could you please retarget this PR to the tp32 branch as I think 
this fix should be applied across all currently maintained versions? Note that 
we have a release coming in the next two weeks so this PR arrived just in time 
- thanks again.


---

Reply via email to