You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/06/24 13:04:34 UTC

[GitHub] [airflow] potiuk commented on a change in pull request #16630: Remove SQLAlchemy <1.4 constraint

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:
users@infra.apache.org