moomindani commented on PR #3630: URL: https://github.com/apache/iceberg-python/pull/3630#issuecomment-5211044112
Not stale — waiting for review. CI is green (17/17) and mergeable. Both of @ebyhr's comments from July 10 are addressed in `818e6f99`: `SortField.__str__` now renders all source ids the same way as `PartitionField.__str__`, and the defensive alias handling was removed. This is a read-path correctness fix — PyIceberg currently raises `Multi argument transforms are not yet supported` on any partition or sort field with multiple `source-ids`, so a v3 table using one cannot be loaded at all. Closes #3628, part of #1818. @ebyhr would you mind confirming the two threads, and @Fokko / @sungwy a review from either of you would be welcome. -- 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]
