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

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

FlorianHockmann commented on code in PR #1677:
URL: https://github.com/apache/tinkerpop/pull/1677#discussion_r895482377


##########
gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Docs/Reference/GremlinVariantsTests.cs:
##########
@@ -79,6 +79,14 @@ public void SerializationGraphson2Test()
 // end::serializationGraphSon[]
         }
 
+        [Fact(Skip = "No Server under localhost")]

Review Comment:
   Not that important, but you could also remove the test 
`SerializationGraphBinaryTest` above now as it isn't used any more in docs.





> Make GraphBinary the default serialization format for .NET and Python
> ---------------------------------------------------------------------
>
>                 Key: TINKERPOP-2723
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2723
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: dotnet, python
>    Affects Versions: 3.5.2
>            Reporter: Florian Hockmann
>            Priority: Minor
>
> .NET and Python have support for GraphBinary at least since 3.5.0 but kept 
> GraphSON 3 by default. It now seems safe to make GraphBinary the default in 
> 3.6.0.
> Mostly as a reminder so we don't forget it: We also need to update the docs 
> accordingly.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to