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

stephen mallette commented on TINKERPOP-1474:
---------------------------------------------

> so is the goal for the Java GLV to have the Vertex class be removed of its 
> Properties methods? 

If we're talking about a distant future for TinkerPop 4.x then, yes, there will 
only be reference vertices/edges. For now, in TinkerPop 3.x I think we've gone 
too far down the path of enabling properties in Java to ever really take them 
away. We can only hide them via TINKERPOP-1701 and encourage users to think 
property-less. I think we just want to align 3.x with the idea of where things 
are headed further in the future. So the discrepancy of properties will 
basically remain between java and python, js, etc, but we will likely shift 
defaults over time to not include properties for Java (with java users having 
the option to go back to including properties if they like as a method of 
backward compatibility). 

> API Alignment Between Java Gremlin Graph Structure and GLVs
> -----------------------------------------------------------
>
>                 Key: TINKERPOP-1474
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1474
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 3.2.2
>            Reporter: Adam Holmberg
>            Assignee: stephen mallette
>
> The current Java GraphSON implementation and that in the Python GLV leave 
> some question about what *should* be returned from a simple traversal like 
> `g.V()`.
> The java implementation presently assumes that properties could be present 
> and returns a DetachedVertex: 
> https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONSerializersV2d0.java#L420-L433
> The python implementation assumes no such thing and returns something more 
> reminiscent of a ReferenceVertex: 
> https://github.com/apache/tinkerpop/blob/master/gremlin-python/src/main/jython/gremlin_python/structure/io/graphson.py#L238-L242
> Is the java version overreaching, and should not expect properties unless a 
> step calls for them (e.g. ` g.V().valueMap()` or `g.V().values('name')`, or 
> should the Python version be expanded?
> Is there something we can do to establish guidelines for this, and align 
> these APIs?



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to