Florian Hockmann created TINKERPOP-1635:
-------------------------------------------
Summary: gremlin-python: Duplicate serialization of element
property in PropertySerializer
Key: TINKERPOP-1635
URL: https://issues.apache.org/jira/browse/TINKERPOP-1635
Project: TinkerPop
Issue Type: Bug
Components: language-variant
Affects Versions: 3.3.0
Reporter: Florian Hockmann
Priority: Trivial
The {{PropertySerializer}} in the current master branch of gremlin-python
serializes the element property twice:
{code}
elementDict = writer.toDict(property.element)
#...
"element": writer.toDict(elementDict)
{code}
This isn't a problem when the element only contains simple types like strings
that aren't serialized by adding a type attribute (like in the unit test for
this component). However, all other types like integers for example are
serialized two times. So an integer will be serialized to:
{code}
{"@type":"g:Int32","@value":{"@type":"g:Int32","@value":1}}
{code}
Full example:
{code}
>>> p = Property("aKey","aValue", Edge(123, Vertex(1), "edgeLabel", Vertex(2)))
>>> writer.writeObject(p)
'{"@type":"g:Property","@value":{"value":"aValue","key":"aKey","element":{"@type":"g:Edge","@value":{"inV":{"@type":"g:Int32","@value":{"@type":"g:Int32","@value":2}},"id":{"@type":"g:Int32","@value":{"@type":"g:Int32","@value":123}},"outV":{"@type":"g:Int32","@value":{"@type":"g:Int32","@value":1}},"label":"edgeLabel"}}}}'
{code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
