This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new e5a88cc75a4 Eliminate serdag loading added in #56422 (#57537)
e5a88cc75a4 is described below
commit e5a88cc75a46b3281940addc5e3bd56b08cd23e7
Author: Ephraim Anierobi <[email protected]>
AuthorDate: Thu Oct 30 14:18:32 2025 +0100
Eliminate serdag loading added in #56422 (#57537)
It was wrong to load the serdag and not use it. The initial idea was
to use the serdag at line 437 but was omitted. Thinking about it now,
it will be faster to only load serdag when there's a TI associated
with the dag version
---
airflow-core/src/airflow/models/serialized_dag.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/airflow-core/src/airflow/models/serialized_dag.py
b/airflow-core/src/airflow/models/serialized_dag.py
index c584ab27d92..1e75e86aad8 100644
--- a/airflow-core/src/airflow/models/serialized_dag.py
+++ b/airflow-core/src/airflow/models/serialized_dag.py
@@ -418,7 +418,6 @@ class SerializedDagModel(Base):
select(DagVersion)
.where(DagVersion.dag_id == dag.dag_id)
.options(joinedload(DagVersion.task_instances))
- .options(joinedload(DagVersion.serialized_dag))
.order_by(DagVersion.created_at.desc())
.limit(1)
)
