[GitHub] [tinkerpop] spmallette commented on a change in pull request #1415: TINKERPOP-2546 Gremlin-Python Client Transport Layer to use AIOHTTP

[GitBox <gi...@apache.org>]

spmallette commented on a change in pull request #1415:
URL: https://github.com/apache/tinkerpop/pull/1415#discussion_r611496213



##########
File path: docs/src/upgrade/release-3.5.x.asciidoc
##########
@@ -494,14 +494,23 @@ See: link:https://issues.apache.org/jira/browse/TINKERPOP-2517[TINKERPOP-2517]
 
 ==== Python 2.x Support
 
-The gremlinpython module no longer supports Python 2.x. Users must use Python 3 going forward. For the most part, from
+The gremlin-python module no longer supports Python 2.x. Users must use Python 3 going forward. For the most part, from
 a user's perspective, there are no specific API changes to consider as a result of this change. It is also worth
 noting that Jython support has been removed and that `gremlin-python` no longer produces a JVM-based artifact. This
 change means that the `GremlinJythonScriptEngine` no longer exists and there is no way to write native Python lambdas.
 All lambdas should be written using `gremlin-groovy` if they are needed.
 
 See: link:https://issues.apache.org/jira/browse/TINKERPOP-2317[TINKERPOP-2317]
 
+==== Python Transportation Layer Rewrite

Review comment:
       This is good - two things:
   
   1. please include the "See: " to link to the JIRA
   2. i think we should clearly state that Tornado support has been removed.




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

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