LuciferYang opened a new pull request, #57103: URL: https://github.com/apache/spark/pull/57103
### What changes were proposed in this pull request? This PR removes four `_LEGACY_ERROR_TEMP_*` error conditions from `error-conditions.json` that are no longer referenced anywhere in the codebase: - `_LEGACY_ERROR_TEMP_1021` - `_LEGACY_ERROR_TEMP_1255` - `_LEGACY_ERROR_TEMP_2106` - `_LEGACY_ERROR_TEMP_3110` ### Why are the changes needed? These are legacy placeholder error conditions. A full-tree scan (`.scala`, `.java`, `.py`, `.sql`, golden `.out` files) confirms none of them are referenced by any main or test code — they are dead entries. Removing them reduces noise in `error-conditions.json` and is a small step toward eliminating the `_LEGACY_ERROR_TEMP_*` placeholders, per the guidance in `common/utils/src/main/resources/error/README.md`. ### Does this PR introduce _any_ user-facing change? No. Since these error conditions are unreachable from any code path, no user-facing error can reference them. ### How was this patch tested? Existing `SparkThrowableSuite` passes (34/34), which validates that `error-conditions.json` is still valid JSON, alphabetically sorted, and satisfies all format/SQLSTATE invariants. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8) -- 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]
