sehajsandhu123 commented on PR #72000: URL: https://github.com/apache/airflow/pull/72000#issuecomment-5425434971
@Dev-iL Thanks for the review, I've added the requested regression test and upstream discussion links. I understand the caution, but this is an opt-in param, off by default. And it isn't really a new one — the convention already exists in the codebase: OdbcHook and MsSqlHook both support sqlalchemy_scheme for per-connection driver choice (pymssql vs pyodbc), and MySqlHook does the same through its client extra. Also, the 7.0.0 notes already give the metadata DB an explicit psycopg2 opt-out via sql_alchemy_conn — hook connections were the only part of this migration without one. Since it's off by default and the strict xfail will tell us when upstream makes it unnecessary, it should be easy to retire too. -- 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]
