konstin commented on PR #62029: URL: https://github.com/apache/airflow/pull/62029#issuecomment-3910810964
The 30s are a read timeout now, meaning it will effectively only fire if no data was transmitted for that period. The current, higher timeout would only be helpful if you expect to have connections that are completely inactive for 30s, then recover to finish the download. (I tried simulating this locally by unplugging the network, but the only effect was that the connection was broken on the OS level and I had the timeout only caused the request to fail faster.) Also note that this timeout was introduced before it was a read timeout, when there was still a total request timeout, where you need the >30s. Now, there is no total request timeout in uv anymore. We don't see any other major project that has adopted uv after that switch was made setting `UV_HTTP_TIMEOUT`. -- 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]
