amitgilad3 commented on code in PR #1717:
URL: https://github.com/apache/iceberg-python/pull/1717#discussion_r1969511896


##########
tests/integration/test_add_files.py:
##########
@@ -229,6 +229,35 @@ def 
test_add_files_to_unpartitioned_table_raises_has_field_ids(
         tbl.add_files(file_paths=file_paths)
 
 
+@pytest.mark.integration

Review Comment:
   Im not sure this test is any different from existing test (but maybe im 
mistaken), the test verifies that the files are correctly added to the table—by 
checking that the manifest counts match the expected values—but it does not 
directly assert that the file additions are processed concurrently. 
   we already have tests that add multiple file



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