[GitHub] [iceberg] RussellSpitzer commented on a change in pull request #1322: ExpireSnapshots executeWith renamed executeDeleteWith

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

RussellSpitzer commented on a change in pull request #1322:
URL: https://github.com/apache/iceberg/pull/1322#discussion_r468720195



##########
File path: api/src/main/java/org/apache/iceberg/ExpireSnapshots.java
##########
@@ -96,7 +96,7 @@
    * @param executorService an executor service to parallelize tasks to delete manifests and data files
    * @return this for method chaining
    */
-  ExpireSnapshots executeWith(ExecutorService executorService);

Review comment:
       Ah I didn't think we hit a release boundary since that addition, sure no problem




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

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