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

ASF GitHub Bot commented on TINKERPOP-3250:
-------------------------------------------

kenhuuu commented on code in PR #3447:
URL: https://github.com/apache/tinkerpop/pull/3447#discussion_r3399124986


##########
gremlin-dotnet/src/Gremlin.Net/Structure/IO/GraphBinary4/RequestMessageSerializer.cs:
##########
@@ -31,6 +31,12 @@ namespace Gremlin.Net.Structure.IO.GraphBinary4
     /// <summary>
     ///     Serializes a <see cref="RequestMessage"/> in the GraphBinary 4.0 
wire format.
     /// </summary>
+    /// <remarks>
+    ///     This serializer is no longer used by the default driver flow 
(which now serializes

Review Comment:
   You should start a thread on the devlist for this.





> Modify default request serialization to JSON
> --------------------------------------------
>
>                 Key: TINKERPOP-3250
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-3250
>             Project: TinkerPop
>          Issue Type: Improvement
>            Reporter: Ken Hu
>            Priority: Major
>
> Changing the default request serialization is more aligned to how most HTTP 
> APIs operate and will allow for easier debugging. This is especially true for 
> applications that will rely on interceptors as it is difficult to see what 
> the serialized bytes are for GraphBinary.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to