denimalpaca commented on a change in pull request #18718:
URL: https://github.com/apache/airflow/pull/18718#discussion_r741945657



##########
File path: airflow/providers/snowflake/operators/snowflake.py
##########
@@ -131,281 +126,3 @@ def execute(self, context: Any) -> None:
 
         if self.do_xcom_push:
             return execution_info
-
-
-class SnowflakeCheckOperator(SQLCheckOperator):

Review comment:
       This PR was opened as an alternative to adding more 
`SnowflakeCheckOperators`, so it seemed to me that their removal here made 
sense given the nature of this change making all these `*Check*` operators 
redundant. Although I see how it could warrant a separate PR.




-- 
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-unsubscr...@airflow.apache.org

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


Reply via email to