@spmallette 
Sorry, I made a mistake in the previous analysis, and now correct it: the  
[Property 
comparison](https://github.com/apache/tinkerpop/blob/be4defb541e9ce99712a53200dbe7794c63f06a3/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/ElementHelper.java#L485)
 issue is not caused by StarEdge, but caused by DetachedProperty. when testing 
query like `g.V().bothE().properties().dedup()` with spark-gremlin, two 
identical instances of DetachedProperty with element=null(both null) are not 
equal, so the dedup() does not work and there are still duplicate properties.

I have submitted a patch to fix the issue: 
https://github.com/apache/tinkerpop/pull/1224/commits/612f81457f15f6577daa724f605c5dcfac87f93b

[ Full content available at: https://github.com/apache/tinkerpop/pull/1224 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to