szehon-ho commented on code in PR #8251: URL: https://github.com/apache/iceberg/pull/8251#discussion_r1289004846
########## docs/spark-procedures.md: ########## @@ -283,11 +283,27 @@ Iceberg can compact data files in parallel using Spark with the `rewriteDataFile | `options` | ️ | map<string, string> | Options to be used for actions| | `where` | ️ | string | predicate as a string used for filtering the files. Note that all files that may contain data matching the filter will be selected for rewriting| +#### Options + +| Name | Default Value | Description | +|------|---------------|-------------| +| `max-concurrent-file-group-rewrites` | 5 | Maximum number of file groups to be simultaneously rewritten | Review Comment: Yea took a second look, you are right, some of them are too long. But now we are removing references to javadoc, I suggest to still add back more context, because some options are a bit complex. I added suggestion to individual options. -- 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]
