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

Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/09/12 06:57:16 UTC

[GitHub] [airflow] potiuk commented on pull request #34293: Update min-sqlalchemy version to account for latest features used

potiuk commented on PR #34293:
URL: https://github.com/apache/airflow/pull/34293#issuecomment-1715110697

   I am not sure if this test will pass here - we might have other functionality used from even later versions of sqlalchemy - but once this PR gets merged we should be protected against making similar accidental SQLAlchemy new features used without bumping the min version. 
   
   SQLAlchemy - unfortunately - does not follow SEMVER and they are adding features in patchlevel versions, so we might accidentally use features (like it was in this case) added at any release. This one should protect us.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org