RussellSpitzer commented on code in PR #7499: URL: https://github.com/apache/iceberg/pull/7499#discussion_r1190345543
########## docs/spark-writes.md: ########## @@ -339,74 +331,61 @@ USING iceberg PARTITIONED BY (days(ts), category) ``` -To write data to the sample table, your data needs to be sorted by `days(ts), category`. - -If you're inserting data with SQL statement, you can use `ORDER BY` to achieve it, like below: +To write data to the sample table, data needs to be sorted by `days(ts), category` but this is taken care +of automatically by the default `hash` distribution. Previously this would have required manually sorting, but this +is no longer the case. Review Comment: The order by wasn't required before either, the "OrderBy" would automatically set the distirbution mode to range. I feel like that was just confusing. -- 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]
