shahar1 opened a new pull request, #70895:
URL: https://github.com/apache/airflow/pull/70895
`SQLColumnCheckOperator` checks declared on a date or text column — `{"min":
{"geq_to": "2020-01-01"}}`, `{"max": {"leq_to": date(2020, 12, 31)}}` — now
fail the task with `TypeError: unsupported operand type(s) for -: 'str' and
'int'`.
#69736 rewrote `_get_match` so the tolerance margin widens the bound outward
for negative expected values, which is correct, but it routed *every* bound
through arithmetic. With no tolerance configured the margin is `0`, and `bound
- 0` still runs — on a bound that cannot be subtracted from. These checks
compared cleanly before.
The same rewrite turned `equal_to` into a degenerate range, so an `equal_to`
check with `accept_none=False` and a NULL result stopped reporting a failed
check and started raising on `None <= bound` instead.
This keeps the abs()-margin semantics when a tolerance is set and compares
the bound directly when it is not.
Verified as a regression: the added tests all pass on 69736's parent commit
and all fail on `main`.
related: #69736
---
##### 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
