danielhumanmod commented on code in PR #518:
URL: https://github.com/apache/incubator-xtable/pull/518#discussion_r1726214209
##########
xtable-core/src/test/java/org/apache/xtable/iceberg/TestIcebergDataHelper.java:
##########
@@ -99,7 +99,9 @@ public class TestIcebergDataHelper {
NestedField.optional(
28, "timestamp_micros_nullable_field",
Types.TimestampType.withZone()),
NestedField.optional(
- 30, "timestamp_local_micros_nullable_field",
Types.TimestampType.withoutZone()));
+ 30, "timestamp_local_micros_nullable_field",
Types.TimestampType.withoutZone()),
+ NestedField.optional(33, "uuid_field", Types.UUIDType.get())
Review Comment:
> Thank you for digging in and providing the context up front of what you
have tested and where the issue is, less homework for me :)
My pleasure :) @the-other-tim-brown , this is a great opportunity for me to
learn more about Spark and Parquet. Additionally, if we determine that this
issue is indeed caused by incomplete support for UUIDs in the Spark Parquet
converter, we could potentially contribute some code to the Spark community.
--
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]
