[
https://issues.apache.org/jira/browse/TINKERPOP-2028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16595266#comment-16595266
]
Kevin Gallardo commented on TINKERPOP-2028:
-------------------------------------------
Re-registering doesn't sound problematic from what I can see.
I'm making a PR for this, will test that.
> AbstractGraphSONMessageSerializerV2d0 should register GremlinServerModule
> when mapper is provided
> -------------------------------------------------------------------------------------------------
>
> Key: TINKERPOP-2028
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2028
> Project: TinkerPop
> Issue Type: Bug
> Components: io
> Affects Versions: 3.3.1, 3.2.9
> Reporter: Kevin Gallardo
> Priority: Major
>
> When specifying a {{GraphSONMapper}} when constructing a
> {{GraphSONMessageSerializerV3d0}}, the Message serializer will not register
> any module. Hence users will have to manually register the
> {{GremlinServerModule}} to the GraphSONMapper they are giving in parameter
> for it to work at all:
> {code:java}
> GraphSONMessageSerializerV3d0 messageSerializerV3d0 =
> new GraphSONMessageSerializerV3d0(
> GraphSONMapper.build()
> .version(GraphSONVersion.V3_0)
> .addCustomModule(new
> AbstractGraphSONMessageSerializerV2d0.GremlinServerModule())
> .addRegistry(TinkerIoRegistryV3d0.instance()).create());
> {code}
> When not providing a Mapper, the GremlinServerModule is registered
> automatically. It sounds reasonable in the context of the
> {{GraphSONMessageSerializer}} to automatically register the
> {{GremlinServerModule}} wdyt?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
