Skip to content

[SPARK-58775][UI] Fix SQL metrics filter and min,med,max table interaction - #57997

Open
wjgan7 wants to merge 3 commits into
apache:masterfrom
wjgan7:spark-sql-viz-metric-filter-fix
Open

[SPARK-58775][UI] Fix SQL metrics filter and min,med,max table interaction#57997
wjgan7 wants to merge 3 commits into
apache:masterfrom
wjgan7:spark-sql-viz-metric-filter-fix

Conversation

@wjgan7

@wjgan7 wjgan7 commented Aug 13, 2026

Copy link
Copy Markdown

What changes were proposed in this pull request?

Change the way filtering metrics is done in spark-sql-viz.js to fix bug with metrics showing blank

Why are the changes needed?

#54887 introduced a filter for SQL metrics. It is very useful, but when a metric has min, med, and max values, it shows the metric as blank

image image

Does this PR introduce any user-facing change?

Yes, it affects the Spark UI

How was this patch tested?

  • Added test in ui-test/
  • Verified locally
image

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Code / Opus 4.8

@wjgan7 wjgan7 changed the title [SPARK-XXXX][UI] Fix filter and min,med,max table interaction [SPARK-XXXX][UI] Fix SQL metrics filter and min,med,max table interaction Aug 13, 2026
@wjgan7 wjgan7 changed the title [SPARK-XXXX][UI] Fix SQL metrics filter and min,med,max table interaction [SPARK-58775][UI] Fix SQL metrics filter and min,med,max table interaction Aug 14, 2026

@uros-b uros-b 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.

Looks good, thank you @wjgan7! Please check the CI failures.
Also, I'm adding @yaooqinn to take a look as well.

@uros-b
uros-b requested a review from yaooqinn August 15, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants