marko-radoicic-db opened a new pull request, #59040: URL: https://github.com/apache/spark/pull/59040
### What changes were proposed in this pull request? This follow-up to #58777 removes the API-dependent `assert_column_resolution_error` helper from the shared column tests. Classic error conditions are asserted inline, while the Connect tests with intentionally different public errors are explicitly overridden and documented in `ColumnParityTests`. This addresses the concern raised in https://github.com/apache/spark/pull/58777#issuecomment-5699678816. ### Why are the changes needed? An inherited Connect parity test normally communicates identical Classic and Connect behavior. Selecting different expected errors through an overridden helper hid observable error-condition differences inside otherwise inherited tests. Explicit Connect overrides make those differences visible and prevent future parity analysis from treating them as identical behavior. ### Does this PR introduce _any_ user-facing change? No. This is a test-only refactoring. ### How was this patch tested? The changed files pass `git diff --check` and an in-memory Python syntax compilation check. Focused PySpark unit tests have not yet been run; this PR is opened as a draft pending that verification. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: OpenAI Codex (GPT-5) -- 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]
