Skip to content

Conversation

@Subham-KRLX
Copy link
Contributor

Remove deprecated TimeDeltaSensorAsync class

Removes TimeDeltaSensorAsync class which was deprecated in favor of the standard TimeDeltaSensor(deferrable=True) pattern used by other Airflow sensors.

Changes:

  • Removed TimeDeltaSensorAsync class and exports
  • Updated tests and documentation to use TimeDeltaSensor(deferrable=True)

closes: #59644


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@jroachgolf84
Copy link
Collaborator

@Subham-KRLX, I think we want this to only be a docs change, correct?

@Subham-KRLX Subham-KRLX force-pushed the remove-timedelta-sensor-async branch from f53d213 to 1934db4 Compare December 20, 2025 03:13
@Subham-KRLX
Copy link
Contributor Author

@fritz-astronomer @jroachgolf84 - Rebased to docs-only change. Added link to deferring docs as suggested and removed all unrelated commits. Ready for review.

@uranusjr uranusjr changed the title Remove timedelta sensor async Remove documentation on TimeDeltaSensorAsync Dec 20, 2025
@uranusjr uranusjr added type:doc-only Changelog: Doc Only and removed provider:amazon AWS/Amazon - related issues area:task-sdk area:core-operators labels Dec 20, 2025
@potiuk
Copy link
Member

potiuk commented Dec 20, 2025

@fritz-astronomer @jroachgolf84 - Rebased to docs-only change. Added link to deferring docs as suggested and removed all unrelated commits. Ready for review.

First make sure all the tests pass.

…rence

- Remove separate TimeDeltaSensorAsync documentation section
- Add reference to official deferring documentation
- Show both sync and deferrable examples using TimeDeltaSensor

Related: apache#59644
@Subham-KRLX Subham-KRLX force-pushed the remove-timedelta-sensor-async branch from 9dfda00 to 248e413 Compare December 20, 2025 15:55
@Subham-KRLX Subham-KRLX force-pushed the remove-timedelta-sensor-async branch from 248e413 to 6710871 Compare December 20, 2025 16:29
Copy link
Collaborator

@jroachgolf84 jroachgolf84 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a committer, but this looks good to me. Thanks for doing this!

@Subham-KRLX
Copy link
Contributor Author

I have synced the branch with main and applied the requested documentation updates: removed the
TimeDeltaSensorAsync
section and added a recommendation for deferrable=True to
TimeDeltaSensor
. This is now a docs-only change. Please let me know if any further updates are needed.

@jscheffl jscheffl merged commit 275d9ba into apache:main Dec 31, 2025
69 checks passed
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
* docs: improve TimeDeltaSensor documentation with deferrable mode reference

- Remove separate TimeDeltaSensorAsync documentation section
- Add reference to official deferring documentation
- Show both sync and deferrable examples using TimeDeltaSensor

Related: apache#59644

* docs: remove TimeDeltaSensorAsync documentation

---------

Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
Co-authored-by: Amogh Desai <amoghrajesh1999@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TimeDeltaSensorAsync vs TimeDeltaSensor(deferrable=True)

7 participants