potiuk commented on code in PR #29044: URL: https://github.com/apache/airflow/pull/29044#discussion_r1082731538
########## airflow/providers/common/sql/hooks/sql.py: ########## @@ -169,6 +169,33 @@ def get_uri(self) -> str: conn.schema = self.__schema or conn.schema return conn.get_uri() + def resolve_rds_cname(self, hostname): Review Comment: Nope. That's not enough. I do not want the AWS code to be in non-AWS provider (even though we have it now in places). We have an AWS provider - all code specific to AWS should be there, if/when we split providers out of Airlfow core, I do not want any external service code in "apache-airlfow" package. The requirement is simple: none of AWS-specific code should be in `apache/airlfow/providers/common/sql` or `apache/airflow`. All the AWS-specific code should be in `apache/airflow/providers/aws/` That's very simple and straighforward expectation. Exactly how this should be done and how the code is to be injected from one package to the other (via callbacks, hooks, etc. ) - I am open to any proposal. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org