You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "moomindani (via GitHub)" <gi...@apache.org> on 2026/09/09 01:50:24 UTC
[PR] Warn about the DatabricksSQLStatementsSensor exclusivity tightening [airflow]
moomindani opened a new pull request, #72761:
URL: https://github.com/apache/airflow/pull/72761
`#70831` moved `DatabricksSQLStatementsSensor`'s exclusivity check from `execute()` to `__init__` and changed it from a truthiness test to `is not None`. That is a behaviour change against a **released** version, because `providers-databricks/7.19.0` (tagged 2026-08-19) already contains #70340 and so ships the `execute()`-time truthiness check:
* `statement="SELECT 1"` with `statement_id=""` runs the statement in 7.19.0, and is a Dag import error now.
* Same for a `statement_id` template that renders to `None` under `render_template_as_native_obj=True` — it is a non-`None` string at construction time, so the pair is rejected at parse time.
* The two argument-validation errors are now `ValueError`, so an `except AirflowException` around them stops matching.
The tightening itself is sanctioned by #70503 and is not in question here. The note explaining it to users was requested during the review of #70831 but did not land before the merge, so the `Changelog` header in this provider still has nothing under it.
The wording is the text written in that review, placed where the file's own header comment asks for it (just below `Changelog`, above the topmost version), in the same shape as the `ComprehendCreateDocumentClassifierOperator` warning at the top of the amazon changelog.
I verified both halves before writing it: `providers-databricks/7.19.0`'s sensor raises `AirflowException` from `execute()` under `if self.statement and self.statement_id`, and `main`'s raises `ValueError` from `__init__` under `if statement is not None and statement_id is not None`. Providers take no newsfragment, so the changelog is the only place this can be said. `check-changelog-format` passes.
related: #70831
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 5)
Generated-by: Claude Code (Opus 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
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
Re: [PR] Warn about the DatabricksSQLStatementsSensor exclusivity tightening [airflow]
Posted by "moomindani (via GitHub)" <gi...@apache.org>.
moomindani commented on PR #72761:
URL: https://github.com/apache/airflow/pull/72761#issuecomment-5595091715
@potiuk this is the changelog note from your review of #70831, placed where the file's header comment asks for it. The wording is yours, unchanged.
---
Drafted-by: Claude Code (Opus 5); reviewed by @moomindani before posting
--
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
Re: [PR] Warn about the DatabricksSQLStatementsSensor exclusivity tightening [airflow]
Posted by "eladkal (via GitHub)" <gi...@apache.org>.
eladkal merged PR #72761:
URL: https://github.com/apache/airflow/pull/72761
--
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