Skip to content

Conversation

sunank200
Copy link
Collaborator

@sunank200 sunank200 commented Jun 26, 2025

This PR refreshed public-airflow-interface.rst for Airflow 3.0+, with the airflow.sdk, deprecating direct metadata DB access in task code, and pointing users to Task Context, REST API, or Python Client.

This patch updates user how-to guides (connections, custom-operator, extra-link) to:
– Add warning against direct metadata-model imports
– Switch examples from airflow.models.* to airflow.sdk.* classes and types

Screenshot 2025-07-02 at 12 11 45 AM

closes: #51590
closes: #52250


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

@potiuk
Copy link
Member

potiuk commented Jun 26, 2025

Actually - I am not sure if we need to make a distinction vs. Airlfow 3 and Airflow 2 - as soon as we have a clearer way of showing that you are in Airflow 2 or Airflow 3, this should be likely more distinguishable. But what I would likely do here is just to focus on Airflow 3 public interface and have a BOLD LETTTERS at the top - somethign like

This is Public interface for Airflow 3 - if you look for Airflow 2 public interface look -> Link to 2.11 version of the interface.

@potiuk
Copy link
Member

potiuk commented Jun 26, 2025

Sorry - public interface of course not security model :) .. My mind was wandering a bit to another similar change we need to do.

@sunank200 sunank200 force-pushed the Update-Public-Interface branch 2 times, most recently from e95f771 to e6fa6ae Compare July 1, 2025 18:29
@sunank200
Copy link
Collaborator Author

Actually - I am not sure if we need to make a distinction vs. Airlfow 3 and Airflow 2 - as soon as we have a clearer way of showing that you are in Airflow 2 or Airflow 3, this should be likely more distinguishable. But what I would likely do here is just to focus on Airflow 3 public interface and have a BOLD LETTTERS at the top - somethign like

This is Public interface for Airflow 3 - if you look for Airflow 2 public interface look -> Link to 2.11 version of the interface.

@potiuk I have done the change as suggested above.

@sunank200 sunank200 force-pushed the Update-Public-Interface branch 2 times, most recently from f3dfd49 to 7cfa2bf Compare July 1, 2025 18:38
@sunank200 sunank200 requested a review from potiuk July 2, 2025 06:52
@sunank200 sunank200 force-pushed the Update-Public-Interface branch from 7cfa2bf to 3f0877e Compare July 2, 2025 12:25
@sunank200 sunank200 requested a review from amoghrajesh July 2, 2025 12:41
@sunank200 sunank200 force-pushed the Update-Public-Interface branch 3 times, most recently from b1f7707 to ca2b9b3 Compare July 2, 2025 15:18
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

I think these are my last set of comments @sunank200.
Looking in pretty good shape!

@sunank200 sunank200 force-pushed the Update-Public-Interface branch 3 times, most recently from 451b4a2 to 6323a86 Compare July 3, 2025 11:41
@sunank200 sunank200 requested a review from amoghrajesh July 3, 2025 16:23
@sunank200 sunank200 force-pushed the Update-Public-Interface branch 2 times, most recently from 2fb6cde to d11695b Compare July 3, 2025 17:17
@sunank200 sunank200 force-pushed the Update-Public-Interface branch 2 times, most recently from dd7ff71 to 2935862 Compare July 8, 2025 07:18
@sunank200 sunank200 force-pushed the Update-Public-Interface branch from 2935862 to f80fbd6 Compare July 8, 2025 07:25
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

Looks good, we can come back at improvements at a later point again.
Please address the one comment I have prior to merge.

@eladkal eladkal added the backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch label Jul 8, 2025
@eladkal eladkal added this to the Airflow 3.0.4 milestone Jul 8, 2025
Copy link
Contributor

@eladkal eladkal left a comment

Choose a reason for hiding this comment

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

Not sure if we can get it to 3.0.3 given the next RC? marked it for 3.0.4 for now
cc @kaxil

@potiuk potiuk modified the milestones: Airflow 3.0.4, Airflow 3.0.3 Jul 8, 2025
@potiuk
Copy link
Member

potiuk commented Jul 8, 2025

Not sure if we can get it to 3.0.3 given the next RC? marked it for 3.0.4 for now cc @kaxil

We have RC5 coming. We can get it in as it is doc-only.

@potiuk potiuk merged commit e8767b6 into apache:main Jul 8, 2025
58 checks passed
Copy link

github-actions bot commented Jul 8, 2025

Backport failed to create: v3-0-test. View the failure log Run details

Status Branch Result
v3-0-test Commit Link

You can attempt to backport this manually by running:

cherry_picker e8767b6 v3-0-test

This should apply the commit to the v3-0-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

potiuk pushed a commit to potiuk/airflow that referenced this pull request Jul 8, 2025
…apache#52297)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst

Co-Authored-By: Amogh Desai <[email protected]>

* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------

Co-authored-by: Amogh Desai <[email protected]>
(cherry picked from commit e8767b6)
potiuk pushed a commit to potiuk/airflow that referenced this pull request Jul 8, 2025
…apache#52297)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst

Co-Authored-By: Amogh Desai <[email protected]>

* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------

Co-authored-by: Amogh Desai <[email protected]>
(cherry picked from commit e8767b6)
potiuk added a commit that referenced this pull request Jul 8, 2025
…#52297) (#53011)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst



* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------


(cherry picked from commit e8767b6)

Co-authored-by: Ankit Chaurasia <[email protected]>
Co-authored-by: Amogh Desai <[email protected]>
kaxil pushed a commit that referenced this pull request Jul 9, 2025
…#52297) (#53011)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst



* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------


(cherry picked from commit e8767b6)

Co-authored-by: Ankit Chaurasia <[email protected]>
Co-authored-by: Amogh Desai <[email protected]>
HsiuChuanHsu pushed a commit to HsiuChuanHsu/airflow that referenced this pull request Jul 10, 2025
…apache#52297)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst

Co-Authored-By: Amogh Desai <[email protected]>

* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------

Co-authored-by: Amogh Desai <[email protected]>
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
…apache#52297)

* docs: Update public interface documentation for Airflow 3.0+ for metadata direct access change

* docs: replace direct metadata model imports in how-to examples with airflow.sdk

* Make Public interface for Airflow 3 and add link for Airflow 2.11

* Fix PR comments

* Update airflow-core/docs/public-airflow-interface.rst

Co-Authored-By: Amogh Desai <[email protected]>

* Fix PR comments

* Remove duplicates and remove RuntimeTaskInstanceProtocol as it is not public

* Fix PR comments

---------

Co-authored-by: Amogh Desai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch kind:documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs on AssetAlias are confusing Doc: Update Public Interface as a result of Airflow 3.0
5 participants