[
https://issues.apache.org/jira/browse/TINKERPOP-2472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17245046#comment-17245046
]
ASF GitHub Bot commented on TINKERPOP-2472:
-------------------------------------------
FlorianHockmann commented on pull request #1369:
URL: https://github.com/apache/tinkerpop/pull/1369#issuecomment-739757074
> From how I understand this change, and assuming this PR builds on the
recent move towards System.Text.Json, this change would allow me to easily
stick to Newtonsoft.Json for the foreseeable future, wouldn't it?
Yes, this targets `master` which is already on `System.Text.Json` which will
be released with 3.5.0. And right, you can stick to `Newtonsoft.Json` with this
change if you implement your own `IMessageSerializer`. You just have to
implement especially the deserialization completely on your own as [the
`GraphSONReader` still uses a
type](https://github.com/apache/tinkerpop/blob/master/gremlin-dotnet/src/Gremlin.Net/Structure/IO/GraphSON/GraphSONReader.cs#L103)
specific to our JSON library (`JsonElement`). We have this issue for quite
some time
([TINKERPOP-1696](https://issues.apache.org/jira/browse/TINKERPOP-1696)) and I
haven't found a way yet to get rid of this.
But if you want to do the deserialization on your own, then this doesn't
affect you.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> GraphBinary support in .NET
> ---------------------------
>
> Key: TINKERPOP-2472
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2472
> Project: TinkerPop
> Issue Type: Improvement
> Components: dotnet
> Affects Versions: 3.4.8
> Reporter: Florian Hockmann
> Priority: Major
>
> Support GraphBinary in Gremlin.Net.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
