denimalpaca commented on PR #24476: URL: https://github.com/apache/airflow/pull/24476#issuecomment-1164870413
> I don't think the location `/providers/core` is appropriate because `core` doesn't really give much information. Maybe we should move all the SQL operators in `airflow.operator` into the providers package at `/providers/sql`? Though, I don't really know why the SQL operators are still in core airflow, so it's something to check There are some issues with having a `providers.sql` because there technically is already one, and one gets added in one of the pre-commit tests for provider packages to make sure there aren't duplicate names. -- 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]
