yaooqinn opened a new pull request, #55861: URL: https://github.com/apache/spark/pull/55861
### What changes were proposed in this pull request? Render the SQL execution detail page header as a single-row DataTable that reuses the same column rendering as the SQL listing page. The summary at the top of `/SQL/execution/?id=N` now shows ID, Query ID, Status, Description, Submitted, Duration, Succeeded Jobs and Error Message in the same shape as the listing page. To avoid duplicating column logic, the shared helpers, the appId resolver, the API base builder and the column factory are extracted into a new `sql-table-utils.js`, sourced by both `AllExecutionsPage` and `ExecutionPage`. The detail-page mode of the column factory: 1. skips truncation and self-links, 2. renders Description / Error Message as `<pre class="sql-cell-pre">` so SQL formatting is preserved, 3. wraps long or multi-line values in a native `<details><summary>` disclosure so the cells start collapsed and stay compact. Parent / Sub Execution links remain on the detail page but move under the new summary table as a small `text-muted` line, only when applicable. ### Why are the changes needed? The SQL execution detail page currently shows an unstructured `<ul>` summary that omits the description and the query id. Users have to bounce back to the listing page to see what a query was about. Reusing the listing-page columns gives a consistent, scannable header on every execution detail page and avoids two parallel renderers drifting over time. ### Does this PR introduce _any_ user-facing change? Yes, the SQL execution detail page header is restyled. The set of metadata fields shown there is the union of what the listing page already shows. Long descriptions and error messages can be expanded inline with the disclosure triangle. ### How was this patch tested? - `build/sbt sql/scalastyle` — clean - `dev/lint-js` — clean - `build/sbt 'sql/testOnly *AllExecutionsPageWithInMemoryStoreSuite'` — 5 / 5 passed; assertion added that the new `sql-table-utils.js` script tag is rendered into the listing page. - Manual browser verification on a local Spark UI: short single-line descriptions render as plain `<pre>`; multi-line / long descriptions and long error messages render as a collapsed `<details>` with a one-line summary; the SQL listing page is unchanged. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: GitHub Copilot CLI 1.0.47 with Claude Opus 4.7 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
