kbendick commented on a change in pull request #4008:
URL: https://github.com/apache/iceberg/pull/4008#discussion_r796050552



##########
File path: site/docs/spark-procedures.md
##########
@@ -190,6 +190,7 @@ the `expire_snapshots` procedure will never remove files 
which are still require
 | `table`       | ✔️  | string | Name of the table to update |
 | `older_than`  | ️   | timestamp | Timestamp before which snapshots will be 
removed (Default: 5 days ago) |
 | `retain_last` |    | int       | Number of ancestor snapshots to preserve 
regardless of `older_than` (defaults to 1) |
+| `max_concurrent_deletes` |    | int       | Size of the thread pool used for 
delete file actions (defaults to null, which deletes files serially in the 
current thread without instantiating a dedicated thread pool) |

Review comment:
       Agreed it's a bit long. Will update.
   
   If we wanted to expand on the details, we could add an additional sentence 
below like `older_than` and `retain_last` are. But I think the shorter 
statement is sufficient.




-- 
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]

Reply via email to