JosephLenton opened a new pull request, #2930:
URL: https://github.com/apache/iceberg-rust/pull/2930
## Which issue does this PR close?
* This adds UUID support to `arrow_schema_to_schema` and
`arrow_schema_to_schema_auto_assign_ids`.
* When they see a field with the extension
`ARROW:extension:metadata=arrow.uuid`, then the field is converted to an
Iceberg UUID type.
- Closes #2929
## What changes are included in this PR?
The visitor used for converting from Arrow to Iceberg has a UUID section
added. This is similar to how it handles `variant`.
## Are these changes tested?
* A new test is added to capture UUID conversion on its own.
* UUID is added to large `arrow_schema_to_schema_auto_assign_ids` test.
## AI Disclosure
AI was used to apply the fix to the visitor, and to add a UUID case to the
test `test_arrow_schema_to_schema_with_field_id`. I built a failing test first
and reviewed it's code to ensure it's output worked.
The `test_arrow_schema_to_schema_should_convert_uuids` test, commit message,
and the bug report, were all written without any AI at all.
--
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]
