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


##########
setup.py:
##########
@@ -399,6 +399,8 @@ def write_version(filename: str = str(AIRFLOW_SOURCES_ROOT 
/ "airflow" / "git_ve
     "wheel",
     "yamllint",
     "aioresponses",
+    # TODO: why??
+    "aiobotocore>=2.1.1",

Review Comment:
   I see that this is already merged, but can we please wait until 3/22 before 
making more deferrable contributions? Recent changes from @potiuk in 
https://github.com/apache/airflow/pull/30144 do alleviate many concerns, but 
there is always a chance that users won't be able to access new Boto 
functionality. We are in the process of sharing the raised concerns internally 
at AWS. However, I do understand that there is no better alternative than using 
aiobotocore and tradeoff is not bad after the CI job separation. cc: @syedahsn 
@o-nikolas 



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