zhengruifeng opened a new pull request, #56183: URL: https://github.com/apache/spark/pull/56183
### What changes were proposed in this pull request? This reverts commit 0f13b94bb8f4f1cdff6ad64d3d27c5995561cab9 (#56106), which added the `with_sql_conf` class decorator in `pyspark.testing.sqlutils` and migrated three parity files to use it. ### Why are the changes needed? On second look the decorator doesn't pull its weight: the parity sites it replaced are already concise enough, and we don't have a follow-up wave of callsites lined up that would amortize the new abstraction. Reverting now keeps the test utilities surface minimal; if a clearer use case emerges later we can reintroduce something targeted. ### Does this PR introduce _any_ user-facing change? No - test infrastructure only. ### How was this patch tested? Existing tests. The revert restores the pre-#56106 `setUpClass` / `tearDownClass` overrides in the three migrated parity files verbatim. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (model: 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]
