Skip to content

[FLINK-39585][runtime] Log execution graph creation failures at ERROR Level#28080

Open
herbherbherb wants to merge 1 commit intoapache:masterfrom
herbherbherb:FLINK-39585-log-execution-graph-creation-error
Open

[FLINK-39585][runtime] Log execution graph creation failures at ERROR Level#28080
herbherbherb wants to merge 1 commit intoapache:masterfrom
herbherbherb:FLINK-39585-log-execution-graph-creation-error

Conversation

@herbherbherb
Copy link
Copy Markdown

@herbherbherb herbherbherb commented May 1, 2026

What is the purpose of the change

Promotes the single failure log in CreatingExecutionGraph#handleExecutionGraphCreation from INFO to ERROR. A non-null throwable here means the AdaptiveScheduler couldn't build/restore the ExecutionGraph and the job is immediately transitioned to FAILED. Therefore, ERROR is the appropriate severity for that root-cause line.

Brief change log

  • CreatingExecutionGraph.java: change the throwable-branch log call from info(...) to error(...).

Verifying this change

This change is a trivial logging-severity adjustment without any test coverage.

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

… severity

In the AdaptiveScheduler's CreatingExecutionGraph state, a non-null
throwable means Flink failed to construct/restore the ExecutionGraph and
will immediately transition the job to FAILED. Logging this root cause at
INFO makes managed-Flink incidents harder to detect and triage, since the
log line that names the actual cause sits at the same severity as routine
lifecycle messages.

Promote the single failure log call in handleExecutionGraphCreation from
info to error. Successful and "waiting for resources" paths remain at
debug, and the surrounding terminal-state logging in AdaptiveScheduler
and Dispatcher is unchanged, so this does not turn normal cancellation
or shutdown into ERRORs.
@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented May 1, 2026

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

Copy link
Copy Markdown
Contributor

@spuru9 spuru9 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions Bot added the community-reviewed PR has been reviewed by the community. label May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-reviewed PR has been reviewed by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants