[ https://issues.apache.org/jira/browse/TINKERPOP-1474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15705116#comment-15705116 ]
ASF GitHub Bot commented on TINKERPOP-1474: ------------------------------------------- Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/490 TINKERPOP-1420 has merged to master. @bjmoor do you want to rebase your PR? note this area of development is still somewhat in flux on master given: https://issues.apache.org/jira/browse/TINKERPOP-1565 Perhaps the best option would be to target this work at the tp32 branch where GraphSON 2.0 is stable. wdyt? > 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, language-variant > 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)