spmallette commented on a change in pull request #1515:
URL: https://github.com/apache/tinkerpop/pull/1515#discussion_r766241105
##########
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:
JIRA is fine. i sort of recall the JS approach and that was sorta forced
in there a bit so i guess we should make a ticket for that too.
--
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]
