marsupialtail commented on code in PR #13385:
URL: https://github.com/apache/arrow/pull/13385#discussion_r922523313
##########
python/pyarrow/_s3fs.pyx:
##########
@@ -133,6 +133,21 @@ cdef class S3FileSystem(FileSystem):
assumed role session will be refreshed.
region : str, default 'us-east-1'
AWS region to connect to.
+ request_timeout : double. unit in seconds. default 3.
+ Socket read timeouts for HTTP clients on Windows. This should be more
+ than adequate for most services. However, if you are transfering large
+ amounts of data or are worried about higher latencies, you should set
+ to something that makes more sense for your use case. For Curl, it's
+ the low speed time, which contains the time in number milliseconds
+ that transfer speed should be below "lowSpeedLimit" for the library
+ to consider it too slow and abort. Note that for Curl this config is
+ converted to seconds by rounding down to the nearest whole second
except
+ when the value is greater than 0 and less than 1000ms. In this case it
is
+ set to one second. When it's 0, low speed limit check will be disabled.
+ Note that for Windows when this config is 0, the behavior is not
specified by Windows.
Review Comment:
updated
##########
python/pyarrow/_s3fs.pyx:
##########
@@ -133,6 +133,21 @@ cdef class S3FileSystem(FileSystem):
assumed role session will be refreshed.
region : str, default 'us-east-1'
AWS region to connect to.
+ request_timeout : double. unit in seconds. default 3.
+ Socket read timeouts for HTTP clients on Windows. This should be more
Review Comment:
updated
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
