ulysses-you opened a new pull request, #57066: URL: https://github.com/apache/spark/pull/57066
### What changes were proposed in this pull request? This PR exposes two additional fields in the SQL REST API (`/api/v1/applications/[app-id]/sql/[execution-id]`): - `modifiedConfigs` on `ExecutionData` -- the SQL configs modified for the execution, sourced from `SQLExecutionUIData.modifiedConfigs`. - `desc` on `Node` -- the physical plan node description, sourced from `SparkPlanGraphNode.desc`. ### Why are the changes needed? The SQL REST API already surfaces plan and metric information, but callers cannot see which configs were modified for a given execution, nor the per-node description string. Exposing these makes the REST API self-sufficient for tooling that inspects executions without scraping the Web UI. ### Does this PR introduce _any_ user-facing change? Yes. Two new fields are added to the SQL REST API response: `modifiedConfigs` (map) on the execution object and `desc` (string) on each plan node. Both default to empty/absent, so existing consumers are unaffected. ### How was this patch tested? Extended `SqlResourceSuite` to populate and assert both fields (node `desc` via the `details = true` cases, and `modifiedConfigs` via `verifyExpectedExecutionData`). All 7 tests in the suite pass. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 4.8) 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
