ryanbordo opened a new pull request, #7300: URL: https://github.com/apache/kyuubi/pull/7300
### Why are the changes needed? Adds an optional parameter to pyhive hive connection class init to allow setting the socket connection timeout. By default, thrift socket connections have no timeout which causes connections to hang indefinitely when something goes wrong. This helps us in [dbt ](https://github.com/dbt-labs/dbt-adapters/blob/1657e9084fcafa0a1bee0b7362c870851c2d173c/dbt-spark/src/dbt/adapters/spark/connections.py#L542 )by allowing for multi-threaded management of connections. Reference thrift lines: https://github.com/apache/thrift/blob/0d18fb2e97a00fc56997fa059d6819e54cdff64b/lib/py/src/transport/THttpClient.py#L130 https://github.com/apache/thrift/blob/0d18fb2e97a00fc56997fa059d6819e54cdff64b/lib/py/src/transport/TSocket.py#L111 ### How was this patch tested? Manually on TSocket path and added unit test ### Was this patch authored or co-authored using generative AI tooling? No -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
