lyndonb-bq commented on a change in pull request #1515:
URL: https://github.com/apache/tinkerpop/pull/1515#discussion_r766308380



##########
File path: 
gremlin-python/src/main/python/gremlin_python/process/graph_traversal.py
##########
@@ -93,12 +98,30 @@ def with_(self, k, v=None):
     def withRemote(self, remote_connection):
         source = self.get_graph_traversal_source()
         
source.traversal_strategies.add_strategies([RemoteStrategy(remote_connection)])
+        self.remote_connection = remote_connection
         return source
 
+    def tx(self):
+        # In order to keep the constructor unchanged within 3.5.x we can try 
to pop the RemoteConnection out of the
+        # TraversalStrategies. keeping this unchanged will allow user DSLs to 
not take a break.
+        # This is the same strategy as gremlin-javascript.
+        # TODO: refactor this to be nicer in 3.6.0 when we can take a breaking 
change

Review comment:
       https://issues.apache.org/jira/browse/TINKERPOP-2664




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to