Skip to content

fix unintended print of dag not found in serialised_dag table - #54972

Merged
rawwar merged 3 commits into
apache:mainfrom
rawwar:kalyan/scheduler_job_runner/fix_serialized_dag_not_found
Sep 17, 2025
Merged

fix unintended print of dag not found in serialised_dag table#54972
rawwar merged 3 commits into
apache:mainfrom
rawwar:kalyan/scheduler_job_runner/fix_serialized_dag_not_found

Conversation

@rawwar

Copy link
Copy Markdown
Contributor

we are directly printing that serialised dag is not found without checking. This PR fixes it.

@boring-cyborgboring-cyborgBot added the area:Scheduler including HA (high availability) scheduler label Aug 27, 2025
@rawwarrawwar changed the title Kalyan/scheduler job runner/fix serialized dag not foundfix unintended print of dag not found in serialised_dag tableAug 27, 2025
@rawwar

Copy link
Copy Markdown
ContributorAuthor

Also pointing to an earlier comment from @ephraimbuddy - https://github.com/apache/airflow/pull/49097/files#r2041591540

I'm not sure if we need this change.

@rawwar
rawwar requested a review from uranusjrAugust 27, 2025 08:48
Comment threadairflow-core/src/airflow/jobs/scheduler_job_runner.py Outdated
Comment threadairflow-core/src/airflow/jobs/scheduler_job_runner.py Outdated
@rawwar
rawwarforce-pushed the kalyan/scheduler_job_runner/fix_serialized_dag_not_found branch from f88f756 to 61c6be6CompareSeptember 15, 2025 00:32
Comment threadairflow-core/src/airflow/jobs/scheduler_job_runner.py Outdated
@rawwar
rawwar requested a review from kaxilSeptember 15, 2025 00:36
@rawwar
rawwar merged commit eec40eb into apache:mainSep 17, 2025
60 checks passed
@rawwar
rawwar deleted the kalyan/scheduler_job_runner/fix_serialized_dag_not_found branch September 17, 2025 02:17
suman-himanshu pushed a commit to suman-himanshu/airflow that referenced this pull request Sep 17, 2025
@kaxilkaxil added this to the Airflow 3.1.0 milestone Sep 18, 2025
kaxil pushed a commit that referenced this pull request Sep 18, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Schedulerincluding HA (high availability) scheduler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rawwar@uranusjr@ephraimbuddy@kaxil