potiuk commented on a change in pull request #16630:
URL: https://github.com/apache/airflow/pull/16630#discussion_r657928052
##########
File path: setup.cfg
##########
@@ -149,8 +149,7 @@ install_requires =
pyyaml>=5.1
rich>=9.2.0
setproctitle>=1.1.8, <2
- # SQLAlchemy 1.4 breaks sqlalchemy-utils
https://github.com/kvesteri/sqlalchemy-utils/issues/505
- sqlalchemy>=1.3.18, <1.4
+ sqlalchemy>=1.3.18
Review comment:
I think lower bounds are pretty meaningless unless we know we need some
feature. They are really only used if someone has a version below that - in
which case it will automatically upgrade the dependency when they upgrade
airflow. I ten'd to leave the lower bounds unchanged unless there is a good
reason to do so.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
