Skip to content

Reduce Dag processor log noise from per-Dag run lookups#69452

Merged
Lee-W merged 1 commit into
apache:mainfrom
astronomer:quiet-run-info-logs
Jul 7, 2026
Merged

Reduce Dag processor log noise from per-Dag run lookups#69452
Lee-W merged 1 commit into
apache:mainfrom
astronomer:quiet-run-info-logs

Conversation

@Lee-W

@Lee-W Lee-W commented Jul 6, 2026

Copy link
Copy Markdown
Member

Why

  • _RunInfo.calculate emitted four INFO-level log lines for every Dag on every parse round.
  • In deployments with hundreds of Dags this floods the Dag processor logs with messages that carry no operator-actionable signal.

What

  • Downgrade the four per-Dag lookup log lines in _RunInfo.calculate (airflow-core/src/airflow/dag_processing/collection.py) from log.info to log.debug:
    • "Getting latest run for partitioned Dag"
    • "Getting latest run for non-partitioned Dag"
    • "got latest run"
    • "no latest run found"
  • No behavioural change beyond log verbosity; the diagnostics remain available at DEBUG.

Was generative AI tooling used to co-author this PR?

If generative AI tooling has been used in the process of authoring this PR, please
change below checkbox to [X] followed by the name of the tool, uncomment the "Generated-by".

  • Yes (please specify the tool below)

Generated-by: [Claude] following the guidelines


  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

_RunInfo.calculate logged at INFO level on every parse round for every
Dag, flooding logs in deployments with hundreds of Dags with messages
that carry no operator-actionable signal. Downgrade these to DEBUG.
@Lee-W Lee-W added the backport-to-v3-3-test Backport to v3-3-test label Jul 6, 2026
@Lee-W Lee-W self-assigned this Jul 6, 2026

@jason810496 jason810496 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, no harm to suppress the logging (I double check the time when there info were introduced, and they are all introduced 6 months ago in 230293c)

@Lee-W Lee-W merged commit 821a079 into apache:main Jul 7, 2026
78 checks passed
@Lee-W Lee-W deleted the quiet-run-info-logs branch July 7, 2026 04:13
@github-actions github-actions Bot added this to the Airflow 3.3.1 milestone Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Hi maintainer, this PR was merged without a milestone set.
We've automatically set the milestone to Airflow 3.3.1 based on: backport label targeting v3-3-test
If this milestone is not correct, please update it to the appropriate milestone.

This comment was generated by Milestone Tag Assistant.

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-3-test PR Link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants