[
https://issues.apache.org/jira/browse/TINKERPOP-1474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15597614#comment-15597614
]
stephen mallette commented on TINKERPOP-1474:
---------------------------------------------
I was a bit surprised that [~dmill] and [~davebshow] weren't interested in
seeing properties in the GLVs.
This issue is so weird. I feel like I've argued for both sides of the fence on
it.
We've determined that Gremlin Server will, as a configuration per request,
return properties or not. We seemed to be going down the path where the default
would be to not return properties as for 3.3.0 and that there would be an
option to include properties per request. So from the server perspective the
data is available if a GLV wanted to provide it.
It does seem incongruent that the java version of {{withRemote}} has the
ability to deal with properties (even if that won't be the future default) but
the other language variants will not. It don't think that there is much more
code to manage to have the properties on those GLV structure classes. I would
think that we would draw the line at these classes being read-only of course.
It's not as though you would implement {{Vertex.addEdge()}} or those types of
methods.
> 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: Bug
> Components: io
> Affects Versions: 3.2.2
> Reporter: Adam Holmberg
>
> 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.3.4#6332)
