[SPARK-43252] Assign a name to the error class _LEGACY_ERROR_TEMP_2016

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.5.0
• Fix Version/s: 4.0.0
• Component/s: SQL
• Labels: pull-request-available starter
• Assignee: Deng Ziming
• Reporter: Max Gekk
• Shepherd: Max Gekk
• Created: 24/Apr/23 09:35
• Updated: 18/Dec/23 00:19
• Resolved: 12/Sep/23 07:34

Description

Choose a proper name for the error class _LEGACY_ERROR_TEMP_2016 defined in core/src/main/resources/error/error-classes.json. The name should be short but complete (look at the example in error-classes.json).

Add a test which triggers the error from user code if such test still doesn't exist. Check exception fields by using checkError(). The last function checks valuable error fields only, and avoids dependencies from error text message. In this way, tech editors can modify error format in error-classes.json, and don't worry of Spark's internal tests. Migrate other tests that might trigger the error onto checkError().

If you cannot reproduce the error from user space (using SQL query), replace the error by an internal error, see SparkException.internalError().

Improve the error message format in error-classes.json if the current is not clear. Propose a solution to users how to avoid and fix such kind of errors.

Please, look at the PR below as examples:

• https://github.com/apache/spark/pull/38685
• https://github.com/apache/spark/pull/38656
• https://github.com/apache/spark/pull/38490

Issue Links

• is a clone of: SPARK-43251 Assign a name to the error class _LEGACY_ERROR_TEMP_2015
• is cloned by: SPARK-43253 Assign a name to the error class _LEGACY_ERROR_TEMP_2017
• links to: [Github] Pull Request #40984 (luizfna)
• links to: [Github] Pull Request #42846 (dengziming)
• links to: GitHub Pull Request #40984
• links to: GitHub Pull Request #42846