stevie9868 commented on PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#issuecomment-2442818473
I have decided to move the test under `integeration/test_writes/test_writes.py` test instead of `tests/table/test_init.py` given that: 1. Many of the existing test fixtures have an S3 prefix in their file locations, which necessitates a Docker setup. While we could create our own fixtures, this would require additional work, which I am not sure if it's still worth to do if we can also test it in the integration test. 2. Most transactions related test are tested within the integration tests, which seems to be more logical and appropriate. -- 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
