eladkal commented on pull request #9844: URL: https://github.com/apache/airflow/pull/9844#issuecomment-658998465
@mik-laj @kaxil correct me if I'm wrong here but if this is merged it may cause issues for future releases of backport packages for users who run older versions of Airflow. This change is based on https://github.com/apache/airflow/pull/9124 which was released in Airflow 1.10.11 so if you merge this PR now any future release of druid/qouble/bigquery in backport package will include this PR causing the changed operators to not work as `airflow.operators.sql` doesn't exist for users who run version < 1.1.11 Since this PR doesn't really introduce new functionality maybe it would be better to wait with this? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
