Skip to content

[Do not merge yet] Airflow data-aware scheduling skill including 3.2 partitioned asset events#169

Open
TJaniF wants to merge 3 commits intoastronomer:mainfrom
TJaniF:main
Open

[Do not merge yet] Airflow data-aware scheduling skill including 3.2 partitioned asset events#169
TJaniF wants to merge 3 commits intoastronomer:mainfrom
TJaniF:main

Conversation

@TJaniF
Copy link

@TJaniF TJaniF commented Mar 23, 2026

Hey :)

With partitioned assets in 3.2 I was refactoring our Learn guides from one asset guide into 4 and thought this is a good chunk of information to turn into a skill, especially because of name coalitions with other data tools using "asset" and Airflow 2 information using "Dataset".

@TJaniF TJaniF marked this pull request as ready for review March 23, 2026 10:49
@@ -0,0 +1,804 @@
# Asset-Based Data-Aware Scheduling in Apache Airflow
Copy link
Contributor

Choose a reason for hiding this comment

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

Frontmatter is missing

@TJaniF TJaniF marked this pull request as draft March 24, 2026 00:57
@TJaniF TJaniF marked this pull request as ready for review March 24, 2026 01:58
@tayloramurphy
Copy link
Contributor

@TJaniF what do you think about folding this into the Authoring Dags skill and putting some of this in a new reference file?

There's already a section in that skill that talks about assets and it can be updated to reference a new file in authoring-dags/reference and we can update the trigger phrase for it to mention asset scheduling.

@TJaniF
Copy link
Author

TJaniF commented Mar 24, 2026

@TJaniF what do you think about folding this into the Authoring Dags skill and putting some of this in a new reference file?

There's already a section in that skill that talks about assets and it can be updated to reference a new file in authoring-dags/reference and we can update the trigger phrase for it to mention asset scheduling.

Happy to do that! :)
I was torn between that and a separate skill like hitl has, but I think it does make sense since some users might need assets but not realize that they do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants