AgenticSpark opened a new pull request, #56929:
URL: https://github.com/apache/spark/pull/56929

   ### What changes were proposed in this pull request?
   
   Backport of #56775 to `branch-4.x`, requested by @MaxGekk in 
https://github.com/apache/spark/pull/56775#issuecomment-4856465190 because the 
original change conflicts on this branch.
   
   Detect unsupported Hive `uniontype<...>` values when converting Hive 
`FieldSchema` types to Spark SQL types and raise a dedicated 
`UNSUPPORTED_HIVE_TYPE` error instead of the generic 
`CANNOT_RECOGNIZE_HIVE_TYPE` parser error.
   
   This is a cherry-pick of the merged master commit c90cad6ad2d. The only 
conflict was in `error-conditions.json`: `branch-4.x` does not have the 
`UNSUPPORTED_HIVE_FUNCTION_TYPE` / 
`UNSUPPORTED_HIVE_METASTORE_VERSION_FOR_JAVA` entries that exist on master, so 
the new `UNSUPPORTED_HIVE_TYPE` entry is placed directly between 
`UNSUPPORTED_GROUPING_EXPRESSION` and `UNSUPPORTED_INSERT`. The Scala changes 
apply unchanged.
   
   ### Why are the changes needed?
   
   Spark SQL does not support Hive union types. Today the failure message comes 
from the parser path and does not clearly identify that the Hive union type is 
unsupported.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. Reading a Hive table column that uses `uniontype<...>` now reports 
`UNSUPPORTED_HIVE_TYPE` with the offending Hive type and column name.
   
   ### How was this patch tested?
   
   Cherry-picked from the merged master commit c90cad6ad2d, which passed CI and 
review as #56775. The production Scala hunks apply unchanged on `branch-4.x`; 
only the `error-conditions.json` entry placement differed and was re-validated 
(valid JSON, alphabetical ordering, one structural token per line). CI here 
runs `HiveClientImplSuite` and the `SparkThrowableSuite` "Error conditions are 
correctly formatted" golden check.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Yes. GitHub Copilot assisted with preparing and validating this change.
   


-- 
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]

Reply via email to