[GitHub] [iceberg] jackye1995 commented on a diff in pull request #5289: AWS: Fix TestS3FileIO prefixDelete test. Fix setup of S3 batch deletion tests

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

jackye1995 commented on code in PR #5289:
URL: https://github.com/apache/iceberg/pull/5289#discussion_r923962203


##########
aws/src/test/java/org/apache/iceberg/aws/s3/TestS3FileIO.java:
##########
@@ -218,8 +219,7 @@ public void testPrefixList() {
   public void testPrefixDelete() {
     String prefix = "s3://bucket/path/to/delete";
     List<Integer> scaleSizes = Lists.newArrayList(0, 5, 1000, 2500);
-
-    scaleSizes.parallelStream().forEach(scale -> {
+    scaleSizes.forEach(scale -> {

Review Comment:
   In that case can we reduce the number of files created for local test, and create another integration test for the same use case with parallel delete?



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