dabla commented on PR #41327: URL: https://github.com/apache/airflow/pull/41327#issuecomment-2438686357
> Also - for reference see this PR from @jscheffl : > > Original PR: #41729 > > It has been closed when all the "split" PRs have been merged) > > Resulting PRs: > > * [AIP-69: Airflow Core adjustments for introduction of Edge Executor #41730](https://github.com/apache/airflow/pull/41730) > * [AIP-69: Breeze adjustments for introduction of Edge Executor #41731](https://github.com/apache/airflow/pull/41731) > * [AIP-69: Adding Empty Edge Provider Package #42046](https://github.com/apache/airflow/pull/42046) > * [AIP-69: Adding Edge Provider DB Models #42047](https://github.com/apache/airflow/pull/42047) > * [AIP-69: Add Executor to Edge Provider #42048](https://github.com/apache/airflow/pull/42048) > * [AIP-69: Add API and Plugin to Edge Provider #42049](https://github.com/apache/airflow/pull/42049) > * [AIP-69: Add CLI to Edge Provider #42050](https://github.com/apache/airflow/pull/42050) > * [AIP-69: Add leftover glue of all pieces to Edge Provider #42051](https://github.com/apache/airflow/pull/42051) > * [ AIP-69: Breeze adjustments for introduction of Edge Executor (#41731) #43139](https://github.com/apache/airflow/pull/43139) > > (Maybe I do not have all of that right - but this is more or less the idea @dabla ) Thanks Jarek, this is indeed a nice split and good example. I was thinking of first creating a PR defining the dialect in the providers manager and defining the types but without actually implementing it. Anyway will try to split it up in as much little PRs as posisble. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
