Skip to content

Conversation

@jedcunningham
Copy link
Member

When the DAG processor resorts its file queue by modification time (e.g., after a bundle refresh), files with pending callbacks could lose their position at the front of the queue. This could delay callback execution (like DAG failure callbacks) if those files happened to have older modification times.

The fix partitions the queue during resort: callback files stay at the front in their original order, while only regular files are sorted by mtime.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Cursor (Clause Opus 4.5)

When the DAG processor resorts its file queue by modification time
(e.g., after a bundle refresh), files with pending callbacks could
lose their position at the front of the queue. This could delay
callback execution (like DAG failure callbacks) if those files
happened to have older modification times.

The fix partitions the queue during resort: callback files stay at
the front in their original order, while only regular files are
sorted by mtime.
@ephraimbuddy ephraimbuddy added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Jan 30, 2026
@ephraimbuddy ephraimbuddy merged commit f5e70fc into apache:main Jan 30, 2026
133 of 134 checks passed
@ephraimbuddy ephraimbuddy deleted the fix_callback_in_queue_sorting branch January 30, 2026 07:52
github-actions bot pushed a commit that referenced this pull request Jan 30, 2026
…61232)

When the DAG processor resorts its file queue by modification time
(e.g., after a bundle refresh), files with pending callbacks could
lose their position at the front of the queue. This could delay
callback execution (like DAG failure callbacks) if those files
happened to have older modification times.

The fix partitions the queue during resort: callback files stay at
the front in their original order, while only regular files are
sorted by mtime.
(cherry picked from commit f5e70fc)

Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jan 30, 2026
…pache#61232)

When the DAG processor resorts its file queue by modification time
(e.g., after a bundle refresh), files with pending callbacks could
lose their position at the front of the queue. This could delay
callback execution (like DAG failure callbacks) if those files
happened to have older modification times.

The fix partitions the queue during resort: callback files stay at
the front in their original order, while only regular files are
sorted by mtime.
(cherry picked from commit f5e70fc)

Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
ephraimbuddy pushed a commit that referenced this pull request Jan 30, 2026
…61232) (#61243)

When the DAG processor resorts its file queue by modification time
(e.g., after a bundle refresh), files with pending callbacks could
lose their position at the front of the queue. This could delay
callback execution (like DAG failure callbacks) if those files
happened to have older modification times.

The fix partitions the queue during resort: callback files stay at
the front in their original order, while only regular files are
sorted by mtime.
(cherry picked from commit f5e70fc)

Co-authored-by: Jed Cunningham <66968678+jedcunningham@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:DAG-processing backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants