jbewing commented on PR #13636: URL: https://github.com/apache/iceberg/pull/13636#issuecomment-3505202690
> I have same issue and thanks for the fix OOC, what's your intended usage of the sort order in the manifest files? I'm assuming to better inform a downstream tool about potential optimizations > What's your plan of this change, do you still plan to drive this to be merged? Sorry, I put this one up and kinda forgot about it and one of my other Iceberg PRs after I got radio silence from a maintainer in another PR after some feedback. Since this code is running perfectly fine in a fork of Iceberg, there wasn't a big motivation to get things upstreamed. However, given that you would find benefit from this (and it also appears like there's another user with the same issue: https://github.com/apache/iceberg/issues/14531), I will find some time next week to update my PR on the current master branch and get this reviewed and integrated! -- 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]
