eladkal commented on issue #27292: URL: https://github.com/apache/airflow/issues/27292#issuecomment-1454873522
I don't fully agree and I don't think it's the same case. `common.sql` added a whole new functionality of a new generic operator. The issues were mostly around the new functionality and not around the new provider by itself. The lesson I learned there is add the new functionality first and give it some time before converting all other providers to use it. Back to the Google case. We are not adding anything new. This is more about re-organizing the existed code. To me it seems that the main reason it's not split is the the [common folder](https://github.com/apache/airflow/tree/main/airflow/providers/google/common) which is being used by almost all google space and it will be hard to break it to individual providers. However this folder is not changing that much. Check the commit history and when it does change most of the commits are about styling -- 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]
