Skip to content

Support native asset partitions in watcher events - #74852

Draft
antonlin1 wants to merge 7 commits into
apache:mainfrom
antonlin1:watcher-native-partition-events
Draft

antonlin1 wants to merge 7 commits into
apache:mainfrom
antonlin1:watcher-native-partition-events

Conversation

@antonlin1

@antonlin1 antonlin1 commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Problem

Currently, when an Asset watcher yields a TriggerEvent, Airflow stores its payload in the asset event’s extra metadata.

The event’s native partition_key field is left unset as TriggerEvent has no field to distinguish it as a top level field. When Airflow prepares scheduling work for partition-aware consumers, it requires that native field. Airflow records the event but logs a missing-partition-key warning and skips creating pending partition work.

Fix

Introduce AssetTriggerEvent with an explicit partition_key, separate from payload metadata. Preserve that key through triggerer serialization and event persistence so partition-aware Dags can schedule correctly.


Was generative AI tooling used to co-author this PR?
  • Yes (Codex, GPT-6)

Generated-by: Codex (GPT-6) following the guidelines

Partition-aware Dags cannot consume native partition scopes from watcher
events when the key is retained only as payload metadata.
Watchers need to identify native partitions without adding asset-specific fields to task resumption and task-end events.
@antonlin1
antonlin1 force-pushed the watcher-native-partition-events branch 2 times, most recently from 13041b9 to 944dce5 Compare October 26, 2026 12:21
@antonlin1
antonlin1 force-pushed the watcher-native-partition-events branch from 944dce5 to 3951dc9 Compare October 26, 2026 12:22

This branch has not been deployed

No deployments
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.

1 participant