j1wonpark opened a new pull request, #59549: URL: https://github.com/apache/spark/pull/59549
### What changes were proposed in this pull request? Validate Spark Connect JDBC connections in `isValid(timeout)` using a lightweight Config RPC. Honor the timeout and reject negative values with `SQLException`. ### Why are the changes needed? The current implementation only checks local state, so it can return `true` after the server has closed the session. ### Does this PR introduce _any_ user-facing change? Yes. `isValid()` now checks the server-side session and returns `false` when validation fails. ### How was this patch tested? Added seven tests. Six failed before the fix; all seven pass after it. All 91 tests across the connection, exception mapping, client, and response validation suites passed. Scala style checks also passed. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: OpenAI Codex (GPT-6) -- 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]
