shubham22 commented on code in PR #30032:
URL: https://github.com/apache/airflow/pull/30032#discussion_r1136104998


##########
airflow/providers/amazon/aws/hooks/base_aws.py:
##########
@@ -42,6 +42,7 @@
 import botocore.session
 import requests
 import tenacity
+from aiobotocore.session import AioSession, get_session as async_get_session

Review Comment:
   > Question to @o-nikolas and the taem - maybe it is already planned or maybe 
you could raise it and push internally to add async support directly in boto 
library? That sounds like something that Amazon will be presurred to do anyway 
I guess? That would be a long-term strategic solution if we know at least that 
it is coming at some point in time.
   
   We discussed this with the Boto team last October when we added deferrable 
operators to our roadmap. Yesterday, we revisited the topic with them. The Boto 
team is already planning to include async functionality in a major release, 
which will eliminate the need for the aiobotocore library entirely. However, 
this release won't be available before 2024 as it will be a major change. This 
is why we decided to use aiobotocore in the short-term as it is the best way 
forward, even though it's not officially endorsed by the AWS Boto team.



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