Skip to content

docs(task-sdk): reorganize landing page into six core sections #52196

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sunank200
Copy link
Collaborator

@sunank200 sunank200 commented Jun 24, 2025

This PR reorganises and updates the content for task-sdk:

  • Landing page (task-sdk/docs/index.rst):
    1. Introduction & Getting Started
    2. Public Interface
    3. DAG Authoring Enhancements
    4. Example DAG References
    5. Concepts
    7. API References

  • Add concepts:
    – Explains the service-oriented Task Execution API Server
    – Compares Airflow 2.x vs 3.x architectures
    – Details task lifecycle: scheduling → execution → heartbeats → XCom communication → retries/teardown
    – Covers parsing & runtime contexts, isolation model, and security considerations

  • Architecture diagrams under task-sdk/docs/img/

Screenshot 2025-06-25 at 3 18 25 AM Screenshot 2025-06-25 at 3 18 34 AM Screenshot 2025-06-25 at 3 18 46 AM Screenshot 2025-06-25 at 3 19 00 AM Screenshot 2025-06-25 at 3 19 08 AM Screenshot 2025-06-25 at 3 19 17 AM Screenshot 2025-06-25 at 3 19 28 AM Screenshot 2025-06-25 at 3 19 35 AM

closes: #49627
related: #51590


^ 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.

@sunank200 sunank200 requested a review from cmarteepants June 24, 2025 21:41
@sunank200 sunank200 force-pushed the task-sdk-docs-concepts branch 2 times, most recently from 092c4dc to eb70f52 Compare June 24, 2025 21:54
@sunank200 sunank200 force-pushed the task-sdk-docs-concepts branch from eb70f52 to 88ef0d3 Compare June 25, 2025 06:26
…Interface, DAG Authoring, Examples, Concepts, API Reference)
@sunank200 sunank200 force-pushed the task-sdk-docs-concepts branch from 88ef0d3 to fd59275 Compare June 25, 2025 17:58
specific language governing permissions and limitations
under the License.

Concepts
Copy link
Contributor

Choose a reason for hiding this comment

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

I don’t think these concepts belong in the task sdk documentation.

These docs should focus solely on the DAG author and interfaces relevant to them.

Broader architectural or system level concepts are better suited for the core Airflow docs.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I can move them to airflow-core. But I wanted to add a separate page for concepts, as it is important in the task-sdk as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

Not really, I do not think it matters for task sdk

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is not directly part of task-sdk, but Airflow has it to make task-sdk work. For example, concepts of Supervisor & Task Runner. Should I move them to the airflow core then?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also, the concept of metadata access

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

I do not have a strong opinion on this, will wait on others for it.

Supervisor + task runner can probably stay but we should certainly not document API server here.

@sunank200 sunank200 requested a review from amoghrajesh June 26, 2025 08:12
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.

Airflow Task SDK Docs
2 participants