[jira] [Commented] (TINKERPOP-3250) Modify default request serialization to JSON

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

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

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


##########
docs/src/upgrade/release-4.x.x.asciidoc:
##########
@@ -32,6 +32,33 @@ complete list of all the modifications that are part of this release.
 
 === Upgrading for Users
 
+==== Request Interceptors

Review Comment:
   As Upgrade Documentation I think this section should be grounded in what we had before a little bit. "When TinkerPop supported websockets prior to 4.0.0, it offered a `RequestInterceptor` interface that allowed <stuff>. With the move to HTTP the notion of the 'interceptor' has shifted...."





> 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)