alexott commented on code in PR #27854:
URL: https://github.com/apache/airflow/pull/27854#discussion_r1030345333
##########
airflow/providers/databricks/operators/databricks_sql.py:
##########
@@ -114,16 +112,17 @@ def __init__(
**client_parameters,
**hook_params,
}
+ self.schema = schema
def get_db_hook(self) -> DatabricksSqlHook:
return DatabricksSqlHook(self.databricks_conn_id, **self.hook_params)
- def _process_output(self, schema, results):
Review Comment:
Here `schema` was a schema of the resulting data, not what we have in
`self.schema` that is the name of the schema/database... That schema was used
to write the files, and it looks like this functionality was broken by earlier
refactoring. But it looks like this functionality isn't used because we didn't
receive complains. I need to think how to re-implement it.
--
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]
