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

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

Github user spmallette commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/351#discussion_r68934451
  
    --- Diff: 
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONMapper.java
 ---
    @@ -60,47 +61,78 @@
         private final boolean normalize;
         private final boolean embedTypes;
         private final GraphSONVersion version;
    +    private final TypeInfo typeInfo;
     
         private GraphSONMapper(final List<SimpleModule> customModules, final 
boolean loadCustomSerializers,
    --- End diff --
    
    Could you please make the constructor match this approach:
    
    
https://github.com/apache/tinkerpop/blob/5395aaeb057e4c163233aa8ace2a2975ea827fe5/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoMapper.java#L165-L172
    
    rather than pass the individual parameters we just pass the `Bulider` 
object. Not sure why this wasn't changed already for `GraphSONMapper`.....


> 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