tinkerpop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From spmallette (GitHub) <git...@apache.org>
Subject [GitHub] [tinkerpop] spmallette commented on issue #1094: TINKERPOP-2195: Add finally block in HttpGremlinEndpointHandler to commit transaction
Date Wed, 10 Apr 2019 11:00:08 GMT
I assume that the additional information you are adding in the custom serializer is something
where you do more traversing on returned objects or gather data from other sources? I'm not
sure a serializer is the best place for that. A serializer should only be about transforming
the data to its network payload form and probably shouldn't contain other logic. 

A custom strategy on the other hand, seems like a better place to keep that sort of logic.
You could "detach" and then add your additional data (nothing says you are restricted to detaching
only in the strategy) Or, if that seems unsuitable for some reason then you should probably
create a custom `Channelizer` and your own implementation of that REST endpoint.

[ Full content available at: https://github.com/apache/tinkerpop/pull/1094 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

  • Unnamed multipart/mixed (inline, None, 0 bytes)
View raw message