li-xiang-db commented on code in PR #71374:
URL: https://github.com/apache/airflow/pull/71374#discussion_r3765299125


##########
providers/databricks/src/airflow/providers/databricks/operators/databricks.py:
##########
@@ -769,6 +777,7 @@ def __init__(
         access_control_list: list[dict[str, str]] | None = None,
         wait_for_termination: bool = True,
         git_source: dict[str, str] | None = None,
+        performance_target: str | None = None,

Review Comment:
   Hi @moomindani , thanks for reviewing, and great catch.
   
   This is known behavior on the Databricks side that we agree isn't ideal. 
However currently it can't just be flipped to a hard error, since existing 
workloads today pass unrecognized values and are silently running on the 
default, rejecting those outright would turn a lot of currently-succeeding runs 
into failures.
   
   So I'll leave it as a pass-through to stay consistent with the other named 
parameters here, and we're already working on improving the behavior from the 
Databricks API side.



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