moomindani commented on PR #3631: URL: https://github.com/apache/iceberg-python/pull/3631#issuecomment-5689402367
@kevinjqliu this is step 3 of your #3840 plan — the maintenance operation that lets users rewrite existing manifests so the legacy `equality_ids: list<long>` gets re-emitted as `int`. Would you be able to review it? Since I last commented here, @hedger9487 contributed the `rewrite_if` predicate (co-authored, credited in the body), which is what makes it possible to target exactly those legacy manifests instead of only what size grouping happens to pick up. Tests went from five to eleven. 21/21 green, mergeable, and the docs subsection is included. -- 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]
