-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Add SQLAlchemy to the dev‑dependency group in providers #60472
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
Add SQLAlchemy to the dev‑dependency group in providers #60472
Conversation
8988895 to
092736a
Compare
|
Do I need to update this airflow/providers/postgres/pyproject.toml Line 108 in 39e52f5
airflow/providers/amazon/pyproject.toml Line 179 in 39e52f5
like |
|
Nice! Thanks
This one should be actually removed -> psycopg2 is already added as required dependency - so we can remove psycopg from dev deps.
This one needs to be changed anyway - because we keep on getting warning about aiobotocore not having boto3 extra. I have some other small fixes to fix so I will fix those. Thanks for notifying @Prab-27 ! |
|
@potiuk Can I remove I tried to find the answe but I couldn't 1 - When we run 2 - I have read this - here 3 What’s the meaning of 4 I don’t exactly remember, but when we changed |
No - we install required dependencies and
So far everything we install requires
Also airflow-ctl has "workspace=true" - whrere do you get it does not have it ? Maybe we missed it somewhere? Some older version did not have it yes - but I think we added it since. Look at https://docs.astral.sh/uv/concepts/projects/workspaces/. Workspace is a way to bring together several distributions that are otherwise separate distributions - to make sure that they all can be installed together with common set of dependencies. Apart of the isolation above we also have to support the case where airflow is installed from a single image - where you can run all components: scheduier, api-server, workers (with providers), triggerer etc. -> and we have to be able to make sure that all those distributions that we want to install together potentially - can in fact be installed - i.e. they have no conflicting dependencies. This is what
Not sure what you are asking for here and what changing |
All concepts are explained very well !! I really appreciate you !! |
* update dev-dependencies in providers * update pyproject.toml in apache-hive
* update dev-dependencies in providers * update pyproject.toml in apache-hive
Related : #59895
Related discussion : here and here
Was generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.