-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Convert remaining providers to the new structure #46045
Comments
I'll work on Zendesk, ydb |
I'm working on Trino, Apprise |
Lets please make sure we merge open PRs for the providers (if we can) before the conversion. |
PR for Weaviate. |
Yeah before attempting to convert a particular provider It would be great to check if there are outsanding PRs. I am going through the PRs from the last few weeks to see if they apply - but if anyone will want to take on a provider - just checking if there are no related PRs before will do the job :) |
I'm working on openai and mongodb |
Unfortunately a log of those result in conflicts if they are not done in sequence - so you will have to rebase those that have conflicts :( |
We really need to batch the providers now - seems that many of them can be fully automated - so I suggest to select a range of those providers (you can provide multiple providers as input to the move script) and submit them together. That will make resolving of the conflict a bit easier. |
Working on sqlite, redis, vertica - #46101 - All tests passed |
Yes I have merged |
There should not really be an issue related to the migration but I'd prefer not batching them. In case there is an issue, reverting the batch would be uglier than individually isolating and debugging |
Working on
|
Yeah. Resolving the conflicts is not that bad and actually the more providers we migrate, the less likely conflicts will be - becase all the conflicting stuff is only when you have adjacent changes in the file - i.e. two providers migrated separatelyt that are "neighbours". I think our aim should be to migrate providers semi-randomly :) |
Working on |
We have now a number of providers converted to the new structure. We have almost 90 of them to be converted. And we have automated scripts that do bulk of the work. This was part of #44511
Process of the conversion of each of the provider is easy - and described in https://github.com/apache/airflow/blob/main/dev/moving_providers/README.md
The devlist announcement: https://lists.apache.org/thread/dzbj5yx5kwpbwyr5yscp4wnlsp6p9v8l
The list below keeps track of the conversion. The list is generated via:
The text was updated successfully, but these errors were encountered: