rambleraptor opened a new pull request, #2114:
URL: https://github.com/apache/iceberg-python/pull/2114
<!--
Thanks for opening a pull request!
-->
<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
Closes #1547
# Rationale for this change
The field `source-ids` is being introduced for Iceberg v3. According to the
spec, it should not be supported by Iceberg v1 + v2. This introduces an error
message to ensure that it cannot be used in those versions.
Most of the code here is to plumb through the format_version to the field to
do the actual version check.
# Are these changes tested?
Unit tests included.
# Are there any user-facing changes?
This does change the current support for `source-ids` on Iceberg v1/v2, but
it shouldn't be supported in the first place.
<!-- In the case of user-facing changes, please add the changelog label. -->
--
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]
