ASF GitHub Bot commented on TINKERPOP-1747:

Github user okram commented on the issue:

    I am not too hip to the Python code anymore, but I do know that adding the 
GraphSON3 support was a little "sketchy". Thus, if this is a Python-esque 
improvement upon my non-Python skills, then great.
    VOTE +1.

> Streamline inheritance for gremlin-python GraphSON serializer classes
> ---------------------------------------------------------------------
>                 Key: TINKERPOP-1747
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1747
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: language-variant
>    Affects Versions: 3.3.0
>            Reporter: David M. Brown
>             Fix For: 3.3.0
> This follows up on TINKERPOP-1427 by cleaning up the code used to create the 
> various gremlin-python GraphSON serializers for GraphSON version 2 and 3. 
> Several small changes can improve code style, reduce redundancy, and clarify 
> when custom readers and writers can be passed to the serializer (generic 
> serializer) vs. the GraphSON version specific serializers. 

This message was sent by Atlassian JIRA

Reply via email to