magic-peach opened a new pull request, #69758:
URL: https://github.com/apache/airflow/pull/69758
Duration and timeout comparisons should use `time.monotonic()` instead of
`time.time()` to be immune to system clock adjustments (NTP, DST, etc.). This
aligns with the project coding standard documented in AGENTS.md.
Changes:
- `databricks_base.py`: Replace `time.time()` with `time.monotonic()` in
Azure metadata cache TTL checks (both sync and async versions)
- `databricks.py` trigger: Replace `time.time()` with `time.monotonic()` in
the polling loop timeout check
- `mixins.py`: Replace `time.time()` with `time.monotonic()` in SQL
statement execution timeout checks, and fix grammar error in comment
("important steps; if a query takes to log" → "important step; if a query takes
too long")
Note: `time.time()` is correctly retained for token expiry comparisons
(lines 337, 378, 640, 1021, 1066) because those compare against wall-clock Unix
timestamps received from the Databricks server.
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Opus 4.7)
Generated-by: Claude Code (Opus 4.7) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
---
Drafted-by: Claude Code (Opus 4.7) (no human review 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
