moomindani opened a new pull request, #3623:
URL: https://github.com/apache/iceberg-python/pull/3623

   Closes #3622 (part of #1551)
   
   # Rationale for this change
   
   Allow upgrading existing tables to format version 3: 
`upgrade_table_version(3)` is no longer rejected, and `next-row-id` is 
initialized to 0 on upgrade, following the spec's "Row Lineage for Upgraded 
Tables" (existing snapshots are left unmodified).
   
   **Draft until #3551 merges**: persisting the upgraded metadata requires V3 
metadata serialization, which #3551 enables. Until then the updated integration 
test (`test_upgrade_table_version`) fails at the metadata write. I'll rebase 
and mark this ready for review once #3551 lands.
   
   ## Are these changes tested?
   
   Yes. A new unit test (`test_update_metadata_upgrade_to_v3`) covers the 
upgrade producing `TableMetadataV3` with `next-row-id=0` including 
re-validation, and the `test_upgrade_table_version` integration test is updated 
to upgrade to v3 (currently blocked on #3551 as noted above).
   
   ## Are there any user-facing changes?
   
   Yes: v2 tables can be upgraded to v3 via `upgrade_table_version(3)` once V3 
metadata serialization (#3551) is available.
   
   This pull request and its description were written by Claude Fable 5.
   


-- 
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]

Reply via email to