gustavoatt opened a new pull request #1323: URL: https://github.com/apache/iceberg/pull/1323
## Summary Fixes https://github.com/apache/iceberg/issues/1138#issuecomment-670717050 by making sure that we can convert from `org.apache.parquet.schema.MessageType` to `org.apache.iceberg.types.Type`. The issue was occurring due to Iceberg not being able to map the `int96` type to an Iceberg Timestamp. Also added an end to end test to verify that we can read through Iceberg API the imported parquet file. While doing this, I noticed that the Iceberg generic parquet reader was reading the timestamp as timestamp without timezone instead of a timestamp with timezone. ## Testing Improved our current unit-test to handle this issue ## Reviewers to: @rdblue cc: @thesquelched @aokolnychyi ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
