kacpermuda commented on code in PR #50392:
URL: https://github.com/apache/airflow/pull/50392#discussion_r2084261934


##########
providers/databricks/src/airflow/providers/databricks/operators/databricks_sql.py:
##########
@@ -106,7 +107,8 @@ def __init__(
         self.catalog = catalog
         self.schema = schema
 
-    def get_db_hook(self) -> DatabricksSqlHook:

Review Comment:
   I'm not sure if it's a breaking change, as the `SQLExecuteQueryOperator` - 
base class of `DatabricksSqlOperator` - has the `get_db_hook()` method that 
returns `self._hook`, so the behavior remains the same. Any way, I can 
overwrite the `get_db_hook` method in Databricks operator as well if needed.



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

Reply via email to