Skip to content

Conversation

@gopidesupavan
Copy link
Member

The dag_version_id assignment is not required here, as its being assigned below. see here:

if unmapped_ti:
dag_version_id = unmapped_ti.dag_version_id
elif dag_version := DagVersion.get_latest_version(task.dag_id, session=session):
dag_version_id = dag_version.id
else:
dag_version_id = None


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

@gopidesupavan gopidesupavan merged commit df2f776 into apache:main Dec 27, 2025
69 checks passed
@gopidesupavan gopidesupavan deleted the remove-unnecessary-dag_version_id-assignment branch December 27, 2025 20:43
chirodip98 pushed a commit to chirodip98/airflow-contrib that referenced this pull request Dec 28, 2025
amoghrajesh pushed a commit to astronomer/airflow that referenced this pull request Dec 29, 2025
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
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.

2 participants