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

Reply via email to