You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "khgould (via GitHub)" <gi...@apache.org> on 2023/06/29 19:24:00 UTC

[GitHub] [airflow] khgould commented on a diff in pull request #32263: Add 'parameters' to DatabricksSQLOperator template_fields

khgould commented on code in PR #32263:
URL: https://github.com/apache/airflow/pull/32263#discussion_r1247019210


##########
airflow/providers/databricks/operators/databricks_sql.py:
##########
@@ -71,7 +71,7 @@ class DatabricksSqlOperator(SQLExecuteQueryOperator):
     """
 
     template_fields: Sequence[str] = tuple(
-        {"_output_path", "schema", "catalog", "http_headers", "databricks_conn_id"}
+        {"_output_path", "schema", "catalog", "http_headers", "databricks_conn_id", "parameters"}

Review Comment:
   Oh I see your newest commit @eladkal: https://github.com/apache/airflow/commit/6b4350e89cd1b3cc66347b31b10337105ccb9907
   
   If this indeed adds all of the template fields from the parent class - great! Sorry for the noise.
   
   



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