MaxGekk commented on PR #56557:
URL: https://github.com/apache/spark/pull/56557#issuecomment-4732122017

   @uros-b Update: the Hive serde nested nanos limitation I mentioned earlier 
is now fixed in 80583a20dea.
   
   Root cause: `HiveInspectors.unwrapperFor(ObjectInspector)` had no Catalyst 
type context, so nanos timestamps nested inside `struct`/`array`/`map` were 
deserialized as micros `Long` instead of `TimestampNanosVal` (the nanos 
special-casing only covered top-level columns), causing a `ClassCastException`.
   
   Fix: added a data-type-aware `unwrapperFor(ObjectInspector, DataType)` that 
produces `TimestampNanosVal` for nanos timestamp types and recurses through 
array/map/struct, used from `OrcFileFormat` for nested fields containing a 
nanos timestamp. The Hive nested/complex test is restored and expanded to cover 
`struct`/`array`/`map` across precisions 7/8/9.


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