The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix/dag-serialization-dep-single-pass has failed.
Run started by GitHub user shahar1 (triggered by shahar1).

Head commit for run:
341bc5c2e9f27ae030c8467967ab564a52e8229a / Shahar Epstein 
<[email protected]>
Perf: fold dep detection into task-serialization loop in serialize_dag

serialize_dag() previously iterated dag.task_dict twice in sequence:
once to serialize tasks, then again to call detect_dependencies on each.
Combine into a single pass so the dict is traversed once.

Benchmark (200 tasks × 20 outlets, warm venv):

  before  ~89 ms min / ~140 ms median
  after   ~93 ms min / ~118 ms median

The end-to-end delta is within noise for this scenario because the real
cost inside the second pass — ensure_serialized_asset encode+decode
roundtrips on each outlet — is unchanged by the loop merge.  The commit
is still worth landing as a code-clarity win (single, linear flow) and
eliminates the redundant dict traversal which does show up in profiles
of larger DAGs with fewer outlets per task.

Report URL: https://github.com/apache/airflow/actions/runs/26630454453

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to