Github user liancheng commented on the pull request:

    https://github.com/apache/spark/pull/8988#issuecomment-145951488
  
    Fixed a bug related to UDT: an exception is thrown when reading Parquet 
files containing UDT values under standard mode. Regression tests are added in 
[`UserDefinedTypeSuite`] [1].
    
    In 1.5 and earlier versions, when reading Parquet files containing UDT 
values, we pass schema containing UDT to `CatalystRowConverter` and translate 
UDTs into corresponding Parquet types there to create value converters. The 
problem is that, `CatalystRowConverter` isn't aware of standard/legacy mode, 
and always generate a Parquet type in legacy format. The last commit fixes this 
issue by expanding UDTs early in `CatalystRowMaterializer`, so that 
`CatalystRowConverter` doesn't need to worry about UDTs anymore.
    
    [1]: 
https://github.com/liancheng/spark/commit/d395bfd0bb73a7522e807e43f80e4307a65001a2#diff-5404bf307e20b3603a661e29a8a9f8fc


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to