the-other-tim-brown commented on code in PR #518:
URL: https://github.com/apache/incubator-xtable/pull/518#discussion_r1726004193
##########
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:
Does Iceberg to Delta work as expected? If only Hudi is broken, we can list
that as a limitation and make a test case for Iceberg to Delta for now.
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 :)
--
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]
