dramaticlly commented on a change in pull request #4052:
URL: https://github.com/apache/iceberg/pull/4052#discussion_r800304969
##########
File path: aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIO.java
##########
@@ -94,6 +106,53 @@ public void deleteFile(String path) {
client().deleteObject(deleteRequest);
}
+ /**
+ * Performs a batch deletion of the given paths. This implementation does
not require
+ * that the passed in paths refer to objects which are in the same bucket.
+ * @param paths poths to delete
Review comment:
nit: I would probably worded a bit differently like `Performs a batch
deletion of given paths grouped by the bucket. Deletion will trigger whenever
configured batch size reached for given bucket or for each bucket at the end.`
also probably typo in param comment? `s/poths/paths`
--
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]
