HyunWooZZ opened a new pull request, #60689: URL: https://github.com/apache/airflow/pull/60689
### Summary This PR enhances the Snowflake SQL API hook by consistently applying the http_timeout_seconds parameter to both synchronous and asynchronous HTTP requests. Also it updates retry logic to consider asyncio.TimeoutError retryable. --- ##### Was generative AI tooling used to co-author this PR? - [X] No (please specify the tool below) --- Details 1. Propagate http_timeout_seconds to requests.Session.request(...) in _make_api_call_with_retries. 2. Configure aiohttp.ClientSession with ClientTimeout(total=...) when http_timeout_seconds is provided. 3. Expand _should_retry_on_error to treat asyncio.TimeoutError as retryable. 4. Refactor request kwargs construction to reduce duplication and centralize timeout behavior. --- Tests - Verified timeout propagation for synchronous HTTP requests. - Checked that async ClientSession is configured with the expected timeout. - Ensured async requests are retried on timeout errors. Sync retry behavior is already extensively covered by existing tests.. So, this PR adds coverage for timeout propagation and async timeout retries. -- 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]
