[GitHub] [iceberg] rdblue commented on a diff in pull request #4902: Spark-3.2: Support Zorder option for rewrite_data_files stored procedure

[GitBox <gi...@apache.org>]

rdblue commented on code in PR #4902:
URL: https://github.com/apache/iceberg/pull/4902#discussion_r906538358


##########
docs/spark/spark-procedures.md:
##########
@@ -264,7 +264,7 @@ Iceberg can compact data files in parallel using Spark with the `rewriteDataFile
 | Argument Name | Required? | Type | Description |
 |---------------|-----------|------|-------------|
 | `table`       | ✔️  | string | Name of the table to update |
-| `strategy`    |    | string | Name of the strategy - binpack or sort. Defaults to binpack strategy |
+| `strategy`    |    | string | Name of the strategy - binpack, sort or zorder. Defaults to binpack strategy |

Review Comment:
   I thought the plan was to add zorder support to sort?



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org