GGraziadei commented on PR #17893: URL: https://github.com/apache/iceberg/pull/17893#issuecomment-5648210382
Addressed in 1d53bf5a5 (rebased on main; the Spark changes now live in 4.2 since 4.1 was copied to 4.2 in the meantime). - `applySortStrategy` now branches explicitly on `Zorder` and `Hilbert` and throws `UnsupportedOperationException` for any other `MultiColumnTerm`, so the dispatch stays closed. - The mixed-terms error now names the term types actually found, so it no longer hardcodes "Zorder and Hilbert" for any two distinct classes; for the existing case the message text is unchanged. - Added the `refs()` `Arrays.asList` -> `ImmutableList` note to the PR description. -- 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]
