zachdisc commented on PR #9731: URL: https://github.com/apache/iceberg/pull/9731#issuecomment-1960673099
## Rev 3 * Fixed styling which caused CI/CD to fail * Changed the clustering column to be a struct referencing the actual columns instead of a contacted string `p1::p2::p3`. Thanks to @ Nandhakumar Saravana for pointing this out, when clustering by a string `a::b::c` spark doesn't know about relative priority/hierarchy. AKA `a::b::c` should be close to `a::b::d`, which it does when clustering by actual columns and not a String literal. Change imminent for `Function`al interface addition too, just running last tests -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
