Skip to content

[SPARK-54534][SQL] Migrate Hive-related legacy error codes to proper error conditions#53244

Closed
ganeshashree wants to merge 2 commits intoapache:masterfrom
ganeshashree:SPARK-54534
Closed

[SPARK-54534][SQL] Migrate Hive-related legacy error codes to proper error conditions#53244
ganeshashree wants to merge 2 commits intoapache:masterfrom
ganeshashree:SPARK-54534

Conversation

@ganeshashree
Copy link

What changes were proposed in this pull request?

This PR migrates three Hive-related legacy error codes to proper error conditions with the INTERNAL_ERROR_ prefix:

  1. _LEGACY_ERROR_TEMP_2186INTERNAL_ERROR_SERDE_INTERFACE_NOT_FOUND

  2. _LEGACY_ERROR_TEMP_2187INTERNAL_ERROR_INVALID_HIVE_COLUMN_TYPE

  3. _LEGACY_ERROR_TEMP_2192INTERNAL_ERROR_INVALID_PARTITION_FILTER_VALUE

Why are the changes needed?

  • Improves error message clarity and consistency
  • Removes legacy error codes as part of ongoing cleanup effort
  • Provides better user experience with more descriptive error messages
  • Follows Spark's error condition naming conventions

Does this PR introduce any user-facing change?

Yes, users will see improved error messages for these three Hive-related error scenarios.

How was this patch tested?

  • Existing tests continue to pass
  • Error message format follows Spark's error condition guidelines
  • All error parameters are properly defined in error-conditions.json

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Sonnet 4.5

@cloud-fan
Copy link
Contributor

@ganeshashree can you fix merge conflicts?

…OLUMN_TYPE

The <message> parameter was redundant as the causedBy exception already
contains the detailed error message. When the exception is displayed,
the inner exception's message was appearing twice - once in the main
error message and again in the cause chain.

This change removes the redundant parameter and only includes the
context-specific information (database and table name) in the error
message template.
@cloud-fan
Copy link
Contributor

thanks, merging to master!

@cloud-fan cloud-fan closed this in 8dd478f Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants