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

ASF GitHub Bot commented on TINKERPOP-1274:
-------------------------------------------

Github user robertdale commented on the issue:

    https://github.com/apache/tinkerpop/pull/351
  
    Some suggestions:
    1. A type is not a class. Call types 'type' or '@type' (not clear the @ is 
necessary since it's in the metadata payload anyway); Not @class. (Otherwise be 
consistent and rename all the Type* classes to be Class*. e.g. 
ClassDeserializer)
    2. Don't use Java types. Use BSON as a reference. It has a nice type system 
and solved most if not all of the type concerns. http://bsonspec.org/spec.html
    3. If space and processing efficiency is a priority, then consider actually 
using BSON.
    4. Alternatively, use an external schema to define types. It could even be 
appended to and a part of the output.


> GraphSON Version 2.0
> --------------------
>
>                 Key: TINKERPOP-1274
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1274
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 3.1.2-incubating
>            Reporter: stephen mallette
>            Priority: Minor
>             Fix For: 3.2.1
>
>
> Develop a revised version of GraphSON that provides better support for 
> non-JVM languages that consume it. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to