ajantha-bhat commented on code in PR #9294:
URL: https://github.com/apache/iceberg/pull/9294#discussion_r1426446921


##########
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##########
@@ -270,7 +270,7 @@ public void orphanedFileRemovedWithParallelTasks() throws 
InterruptedException,
 
     waitUntilAfter(System.currentTimeMillis());
 
-    Set<String> deletedFiles = Sets.newHashSet();
+    Set<String> deletedFiles = ConcurrentHashMap.newKeySet();

Review Comment:
   a) Can you please handle in other spark versions too? Flaky test can happen 
from any spark module.
   
   b) Please also mention that it fixes 
   https://github.com/apache/iceberg/issues/8824



-- 
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-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to