potiuk commented on PR #44210: URL: https://github.com/apache/airflow/pull/44210#issuecomment-2489921730
Nope. This is still very wrong. "common" sql provider should not know anything about "oracle" - this is the main reason why we introduced "common.sql" and all the "database" specific calls should come to the provider - so you have to modify it in the way that all "oracle" specific code lives in the "oracle" provider. You should implement a new feature in the common.sql where you can replace the template and then make oracle provider to replace it. -- 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]
