spmallette commented on a change in pull request #1515:
URL: https://github.com/apache/tinkerpop/pull/1515#discussion_r765792310
##########
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:
Are you expecting a simple refactoring here? Or perhaps more the point,
once we merge this, will it be easy for you to submit a follow-on PR to
`master` to do this or will we need to create JIRA for future consideration?
--
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]
