potiuk commented on PR #23915: URL: https://github.com/apache/airflow/pull/23915#issuecomment-1147479264
> Question I have for reviewers: would it make sense to add the Snowflake, Bigquery, Redshift, etc... versions of this operator in with this PR, or open a separate issue & PR for the provider updates? Definitely separate. Also to note - those classes will not be available in Airlfow's core until 2.4 release and our providers are 2.1 (and soon 2.2-compatible). So when you add it to providers, it has to be done in the way that the proivders are importable in 2.2 as well and that those new opertors are treated as optional and provide the right message when someone tries to use the operator in Airflow 2.1 -- 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]
