You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2022/06/10 20:59:56 UTC

[GitHub] [tinkerpop] vkagamlyk commented on pull request #1677: TINKERPOP-2723 Make graphbinary default in gremlin-python and dotnet

vkagamlyk commented on PR #1677:
URL: https://github.com/apache/tinkerpop/pull/1677#issuecomment-1152734465

   > The code changes look good to me, but could you please also update the docs and add a note to the upgrade docs and a changelog entry as this is a breaking change?
   > 
   > [This section about configuration for gremlin-python](https://tinkerpop.apache.org/docs/current/reference/#gremlin-python-configuration) currently mentions `GraphSONMessageSerializer` as the default serializer and [this section shows for Gremlin.Net how to change the default serializer](https://tinkerpop.apache.org/docs/current/reference/#gremlin-dotnet-serialization) from GraphSON 3 to GraphBinary / GraphSON 2.
   > 
   > (I've added a link to the JIRA issue to the PR description.)
   
   Doc's updated


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

To unsubscribe, e-mail: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org