xiazcy commented on code in PR #2283:
URL: https://github.com/apache/tinkerpop/pull/2283#discussion_r1358910305
##########
gremlin-python/src/main/python/gremlin_python/driver/client.py:
##########
@@ -45,15 +46,20 @@ def __init__(self, url, traversal_source,
protocol_factory=None,
kerberized_service="", headers=None, session=None,
enable_user_agent_on_connect=True, **transport_kwargs):
log.info("Creating Client with url '%s'", url)
+
+ # check via url that we are using http protocol
+ self._use_http = re.search('^http', url)
+
self._closed = False
self._url = url
self._headers = headers
self._enable_user_agent_on_connect = enable_user_agent_on_connect
self._traversal_source = traversal_source
- if "max_content_length" not in transport_kwargs:
+ if not self._use_http and "max_content_length" not in transport_kwargs:
transport_kwargs["max_content_length"] = 10 * 1024 * 1024
if message_serializer is None:
message_serializer = serializer.GraphBinarySerializersV1()
+ # message_serializer = serializer.GraphSONMessageSerializer()
Review Comment:
Also something I forgot to remove, thanks!
--
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]
