Github user newkek commented on the issue:
https://github.com/apache/tinkerpop/pull/499
> If we do this, then we would have to change Edge back to the same model
to be consistent
Indeed.
> why change it back?
Because currently it is not consistent,
(Detached)Edge/Vertex/VertexProperty all derive from (Detached)Element, for
which `properties` have the same structure, but with what's currently on the
PR, Edge and VertexProperty don't have their `properties` serialized in the
same format than Vertex.
If we unify all we can make generalize the deserialization of an object
that derives from Element and make the deserialization code more readable.
In my opinion I don't believe verbosity should be chosen over consistency
here, especially since GraphSON 2 is already very very verbose and not really
meant to be compact.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
