[SPARK-56049][UI] Add search/filter for metrics in SQL plan viz side panel#54887
[SPARK-56049][UI] Add search/filter for metrics in SQL plan viz side panel#54887yaooqinn wants to merge 1 commit intoapache:masterfrom
Conversation
…panel Add a search input to the SQL plan visualization side panel that filters metrics by name. Appears when a node has more than 5 metrics (e.g., Exchange with 20+ shuffle metrics). Typing filters rows in real-time by matching the metric name column. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dongjoon-hyun
left a comment
dongjoon-hyun
left a comment
There was a problem hiding this comment.
+1, LGTM. Thank you, @yaooqinn .
|
BTW, are you going to use your personal message instead of complying the Apache Spark Pull Request template, @yaooqinn ? 
I'm not sure why the Copilot ignores the pull request template, but when I ask 
spark/.github/PULL_REQUEST_TEMPLATE Lines 55 to 56 in ca60558 |
|
Thank you @dongjoon-hyun, update the desc |
|
Thank you so much! |
|
I verified manually. Merged to master. Thank you again, @yaooqinn . 
|
|
Thank you @dongjoon-hyun |
…panel ### What changes were proposed in this pull request? Add a search input to the SQL plan visualization side panel that filters metrics by name in real-time. - Appears when a node has **more than 5 metrics** (e.g., Exchange nodes with 20+ shuffle metrics) - Typing filters table rows by matching the metric name column - Instant filtering — no debounce needed for small tables ### Why are the changes needed? Exchange and WholeStageCodegen nodes can have 20+ metrics. Scrolling through all of them to find a specific metric (e.g., "spill", "duration", "data size") is tedious. The search box lets users type a keyword to instantly filter to relevant metrics. ### Does this PR introduce _any_ user-facing change? Yes — a "Filter metrics..." search box appears in the side panel when a node has many metrics. ### How was this patch tested? Manual testing. JS-only change, no Scala changes. ### Was this patch authored or co-authored using generative AI tooling? Yes, Generated-by: GitHub Copilot(claude-opus-4.6-1m). Closes apache#54887 from yaooqinn/SPARK-56049. Authored-by: Kent Yao <kentyao@microsoft.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
2 participants
What changes were proposed in this pull request?
Add a search input to the SQL plan visualization side panel that filters metrics by name in real-time.
Why are the changes needed?
Exchange and WholeStageCodegen nodes can have 20+ metrics. Scrolling through all of them to find a specific metric (e.g., "spill", "duration", "data size") is tedious. The search box lets users type a keyword to instantly filter to relevant metrics.
Does this PR introduce any user-facing change?
Yes — a "Filter metrics..." search box appears in the side panel when a node has many metrics.
How was this patch tested?
Manual testing. JS-only change, no Scala changes.
Was this patch authored or co-authored using generative AI tooling?
Yes, Generated-by: GitHub Copilot(claude-opus-4.6-1m).