hussein-awala commented on PR #30784: URL: https://github.com/apache/airflow/pull/30784#issuecomment-1666906282
@potiuk This PR is ready and fully b/c: - If we use an old version of sub providers with new version of common SQL -> BaseSqlOperator will not find the new attribute `conn_id_field` in the sub class and it will fallback to the default one `conn_id` which is provided as param. - If we use a new version of sub provider with old version of common SQL -> `conn_id_field` will be ignored, and the BaseSqlOperator will use the provided param. - If we use new versions for both -> the provided param `conn_id` will be ignored, and the BaseSqlOperator will get the attr provided in `conn_id_field`, and the issue will be fixed. -- 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]
