-
Notifications
You must be signed in to change notification settings - Fork 15.6k
docs: clarify AssetAlias usage with detailed explanation and examples #52259
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Added a note under "Using Airflow Public Interfaces" to recommend using `airflow.sdk` as the official interface from Airflow 3.0. - Referenced AIP-72 and linked related documentation. - Encouraged users to prefer REST API and Python Client for integrations.
Great Co-authored-by: Jarek Potiuk <[email protected]>
@@ -28,6 +28,21 @@ and for automating certain aspects of the Airflow workflow. | |||
Using Airflow Public Interfaces | |||
=============================== | |||
|
|||
.. note:: | |||
|
|||
As of **Airflow 3.0**, users should use the ``airflow.sdk`` namespace as the official **Public Interface**, as defined in `AIP-72 <https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-72+Task+Execution+Interface+aka+Task+SDK>`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the motivation to mention AIP-72? Generally speaking AIP numbering is internal information of Airflow developers they are not so meaningful to users (we do mention them sometimes but mostly for future plans not as reference to completed work)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire file is also being reworked as part of #52297 by @sunank200
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah cool
so if it suppress this PR lets close it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yea
- `Release Notes 3.0 <https://airflow.apache.org/docs/apache-airflow/stable/release_notes.html>`_ | ||
- `Task SDK Overview <https://airflow.apache.org/docs/apache-airflow/stable/concepts/taskflow.html>`_ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be relative links rather than http ones. We use http only for external resources
Superseded by #52297 |
This PR improves the documentation for
AssetAlias
in the authoring and scheduling assets guide.AssetAlias
is and why it’s useful.outlet_events
andMetadata
.These improvements aim to help users better understand and effectively use
AssetAlias
for dynamic asset management and DAG dependency configuration.Closes: #52250