You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/06/06 14:00:20 UTC

[GitHub] [airflow] potiuk commented on pull request #23915: Add new SQLCheckOperators

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org