dabla commented on code in PR #42891:
URL: https://github.com/apache/airflow/pull/42891#discussion_r1800551291
##########
providers/src/airflow/providers/common/sql/operators/sql.py:
##########
@@ -145,13 +145,25 @@ def __init__(
self.hook_params = hook_params or {}
self.retry_on_failure = retry_on_failure
+ @classmethod
Review Comment:
> NEW_VERSION_OF_COMMON_SQL
I made the standard provider dependent of latest common sql provider, but
this won't solve the issue of Airflow 2 with the get_hook method of BaseHook,
which resides in the airflow core basecode right? So I still need the backward
compatibility I would say until standard provider depends on Airflow 3.0.0 or
higher. Anyway I created a test case for this not to forget as mentioned above.
--
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]
