Taragolis commented on code in PR #28874: URL: https://github.com/apache/airflow/pull/28874#discussion_r1067864413
########## airflow/providers/apache/hive/provider.yaml: ########## @@ -56,6 +56,7 @@ dependencies: # the sasl library anyway (and there sasl library version is not relevant) - sasl>=0.3.1; python_version>="3.9" - thrift>=0.9.2 + - impyla Review Comment: This unfortunetly not transform sync code into asyncio. If this kind transformation would be so easy than we had to transform any sync method to asyncio implementation. So offhand most of `impyla` methods makes blocking io requests: when you call `cursor.execute_async` as well as `cursor.is_executing()` and also `cursor.fetchall()` -- 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]
