tinkerpop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TINKERPOP-2028) AbstractGraphSONMessageSerializerV2d0 should register GremlinServerModule when mapper is provided
Date Wed, 05 Sep 2018 14:07:00 GMT

    [ https://issues.apache.org/jira/browse/TINKERPOP-2028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16604446#comment-16604446

ASF GitHub Bot commented on TINKERPOP-2028:

Github user newkek commented on the issue:

    The reason this is targeting master is because although I did not break backward compatibility
and put previous outdated methods as Deprecated I wasn't sure what the strategy is here for
retiring things in patch versions. Also there would be a slight change of behavior if anybody
were to have also extended `AbstractGraphSON2MessageSerializer`. Let me know if that's acceptable
to put in a patch or not, if so I don't any other reason not to do it in tp33.

> 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

View raw message