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


##########
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:
   > That is also a possibility of course. The end result is though, that if in 
the "sync" package you would like to use some new feature you technically 
should use ">=version_when_it_was_added".
   
   I agree. I don't have a good answer here. This is a dilemma.
   any course of action taken will have pros and cons.
   
   I'm fine with either way we go. We raised the questions I think now we 
should leave it to AWS folks to debate and let us know their chosen path (in 
the spirit of shared governance model)



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