Skip to content

[FLINK-22543][runtime-web] Fix Exception History table layout broken by long error label strings#28151

Merged
rkhachatryan merged 1 commit into
apache:masterfrom
Izeren:FLINK-22543/fix-exception-history-table-layout
May 18, 2026
Merged

[FLINK-22543][runtime-web] Fix Exception History table layout broken by long error label strings#28151
rkhachatryan merged 1 commit into
apache:masterfrom
Izeren:FLINK-22543/fix-exception-history-table-layout

Conversation

@Izeren
Copy link
Copy Markdown
Contributor

@Izeren Izeren commented May 13, 2026

What is the purpose of the change

On the Job > Exceptions > Exception History tab, long error labels cause the Time column to collapse to near-zero width. Text wraps character-by-character vertically, making the entire row unreadable. The table also extends beyond the viewport with no horizontal scrollbar.

Similar issue was reported in FLINK-22543 where Flink SQL task names exacerbate the problem.

Brief change log

  • Set nzTableLayout="fixed" on the Exception History <nz-table> so columns respect their declared widths regardless of content
  • Add percentage-based column widths for Time (12%), Task Name (18%), Location (12%), and the expand toggle (4%); Exception takes the remaining ~54%
  • Add word-break: break-all on the exception cell to force mid-word line wrapping for long unspaced strings
  • Style nz-tag elements inside the exception cell with max-width: 100% and white-space: normal to prevent tag overflow
  • Change .exception-select from fixed width: 300px to width: 100% so the dropdown fills available cell space under fixed layout

Verifying this change

Manually verified with a minicluster producing long exception strings at multiple viewport sizes (4K 27", 16" laptop, heavily squashed window).

Example before the fix:
image

Examples after the fix:

27 inch: (width="2288" height="810")

image

16 inch: (width="1448" height="946")
image

somewhat squashed: (width="610" height="804")
image

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): no
  • The public API, i.e., is any changed class annotated with @Public(Evolving): no
  • The serializers: no
  • The runtime per-record code paths (performance sensitive): no
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
  • The S3 file system connector: no

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 — Claude Code (Claude Opus 4.6)

Generated-by: Claude Code (Claude Opus 4.6)

…by long exception strings

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Izeren Izeren changed the title [FLINK-22543][runtime-web] Fix Exception History table layout broken by long exception strings [FLINK-22543][runtime-web] Fix Exception History table layout broken by long error label strings May 13, 2026
@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented May 13, 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 — scoped, well-verified across viewport sizes. Thanks for the screenshots.

@github-actions github-actions Bot added the community-reviewed PR has been reviewed by the community. label May 14, 2026
Copy link
Copy Markdown
Contributor

@rkhachatryan rkhachatryan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the fix!

@rkhachatryan rkhachatryan merged commit 1e6a736 into apache:master May 18, 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