mderoy opened a new pull request, #46506:
URL: https://github.com/apache/arrow/pull/46506

   ### Rationale for this change
   In extreme cases, a slow network response response from S3, Minio, etc will 
result in arrow returning curl 28 timeout errors even before the 
request_timeout period has elapsed. This is because there is an additional 
setting used in the awssdk known as the lowSpeedLimit (used in the curl 
library) to abort the transfer if the transfer rate drops below some value (by 
default 1 byte / second). In such a case the user may want to disable this 
lowSpeedLimit to allow their reads to succeed despite the slow network 
throughputs.
   
   ### What changes are included in this PR?
   add the low_speed_limit option to S3Options, and then include them in the 
client_config_ if they are >=0
   -1 will use the default setting from the s3 sdk (like other options)
   0 disables the lowSpeedLimit
   >0 will set the lowSpeedLimit to N bytes / s
   
   ### Are these changes tested?
   They are tested our fork in arrow 13.0.0.0 but I have not tested with the 
latest code, as I'm unable to build main in my environment (it looks like a new 
build dependency on ninja was added and I'm not able to install it at the 
moment on my system). I have opened this PR via a clean cherry-pick of my 
changes.
   
   ### Are there any user-facing changes?
   No, these settings are for advanced users of the C++ sdk with S3FS


-- 
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