anoopj opened a new pull request, #2959: URL: https://github.com/apache/iceberg-rust/pull/2959
## Which issue does this PR close? Refs https://github.com/apache/iceberg-rust/issues/2879 ## What changes are included in this PR? The two v3 row-lineage metadata columns were defined as required, but the spec requires readers to produce null for both in legit cases. e.g. when a data file has a null first_row_id, and for added or modified rows before inheritance. Iceberg Java marks both NestedField.optional accordingly. I think this was originally added as required due to an oversight. Mark both fields optional to match. This is a prerequisite for materializing the columns on read, where a null result must be representable. ## Are these changes tested? Added a test -- 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]
