syedahsn commented on code in PR #30127:
URL: https://github.com/apache/airflow/pull/30127#discussion_r1140573922
##########
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 think shipping aiobotocore by default is the better option. It will be a
smoother user experience if users don't have to change their workflows to use
deferrables if their workflows use features not covered by aiobotocore. It's
not an ideal situation in either case, but this option is less likely to cause
issues for users wanting to use deferrable operators.
--
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]
