-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Cleanup some dependencies #60992
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Cleanup some dependencies #60992
Conversation
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
|
cc: @Prab-27 -> should fix some of the things you noticed yesterday in #60472 (comment) |
jscheffl
left a comment
There was a problem hiding this 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 | |||
There was a problem hiding this comment.
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'", |
There was a problem hiding this comment.
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
Prab-27
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool !! LGTM !
There were a few things raising warnings during uv sync that should be fixed:
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.