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

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

Github user jorgebay commented on the issue:

    https://github.com/apache/tinkerpop/pull/842
  
    Sorry I'm late to the party, I haven't got much time during the past weeks: 
nice contribution @danielcweber !
    
    I'm a little concerned about the difference between serializers and 
deserializers supported.
    
    Can we limit the scope of this PR to types for which we support 
serialization and deserialization?
    Some of the extended types are not very popular but once we add support for 
them, the user base for those could grow. From the user perspective, retrieving 
and storing are expected to be symmetric: if one operation is supported the 
other should also be.
    Mapping 1 type (like `DateTime`) to multiple TinkerPop types could lead to 
obscure serialization errors that the user might not be able to identify.



> Add support for extended GraphSon types to Gremlin.net
> ------------------------------------------------------
>
>                 Key: TINKERPOP-1945
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1945
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: dotnet
>            Reporter: Daniel C. Weber
>            Priority: Minor
>
> While Gremlin.net can properly serialize and deserialize all of the core 
> GraphSon types, support for most of the extended types as described 
> [here|[http://tinkerpop.apache.org/docs/3.3.2/dev/io/#_extended]] is lacking.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to