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/10/17 18:41:19 UTC

[GitHub] [airflow] eladkal commented on pull request #25717: Add SQLExecuteQueryOperator

eladkal commented on PR #25717:
URL: https://github.com/apache/airflow/pull/25717#issuecomment-1281315626

   > I like the change - however completely removing the `execute` feels really aggressive. Shouldn't we deprecate notice and then remove later? This will break a _lot_ of workflows if anyone's dependency get's bumped without them knowing about the change.
   
   This PR is backward compatible.
   The operators are now inheriting from `SQLExecuteQueryOperator` so they use this operator `execute()`
   


-- 
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