MaxGekk opened a new pull request #35272: URL: https://github.com/apache/spark/pull/35272
### What changes were proposed in this pull request? In the PR, I propose to switch from specific error classes in the AES functions: `aes_encrypt()/aes_decrypt()` to more generic: - AES_CRYPTO_ERROR -> INVALID_PARAMETER_VALUE - INVALID_AES_KEY_LENGTH -> INVALID_PARAMETER_VALUE - UNSUPPORTED_AES_MODE -> UNSUPPORTED_MODE The new error classes `INVALID_PARAMETER_VALUE` and `UNSUPPORTED_MODE` are made to be re-used from other functions but not only in the AES functions. ### Why are the changes needed? 1. To prevent unlimited inflation of the set of error classes and as a consequence of that inflation of `error-classes.json`. 2. To establish rules for other sub-tasks of SPARK-37935 ### Does this PR introduce _any_ user-facing change? Yes, but the AES functions `aes_encrypt()`/`aes_decrypt()` haven't released yet. ### How was this patch tested? By running the affected test suites: ``` $ build/sbt "test:testOnly *SparkThrowableSuite" $ build/sbt "test:testOnly *QueryExecutionErrorsSuite" ``` -- 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]
