Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jan 23, 2026

There were a few things raising warnings during uv sync that should be fixed:

  • no lower limits on few dependencies
  • old pcycopg2 dev dependency for postgres provider.
  • botocore/aiobotocore depsndencies not updated for a while
  • lack of quotes in some sqlalchemy deps
  • optional deps for sqlalchemy not sorted properly

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

There were a few things raising warnings during uv sync that should
be fixed:

* no lower limits on few dependencies
* old pcycopg2 dev dependency for postgres provider.
* botocore/aiobotocore depsndencies not updated for a while
* lack of quotes in some sqlalchemy deps
* optional deps for sqlalchemy not sorted properly
@potiuk
Copy link
Member Author

potiuk commented Jan 23, 2026

cc: @Prab-27 -> should fix some of the things you noticed yesterday in #60472 (comment)

Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

@@ -67,8 +67,8 @@ dependencies = [
# in the `devel-dependencies` section to be the same minimum version.
# Apparently latest boto is causing OOM (143) issues in our CI
# we limit it for now - see https://github.com/apache/airflow/issues/49937
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we remove this comment ? since there's no limit here - (related issue)

"apache-airflow-providers-common-compat>=1.12.0",
"apache-airflow-providers-common-sql>=1.23.0",
"psycopg2-binary>=2.9.9; python_version < '3.13'",
"psycopg2-binary>=2.9.10; python_version >= '3.13'",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we upgrade this deps since we have SQLAlchemy2.0 ? related discussion : here

Copy link
Contributor

@Prab-27 Prab-27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool !! LGTM !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants