jorisvandenbossche commented on code in PR #39825: URL: https://github.com/apache/arrow/pull/39825#discussion_r1507226334
########## python/pyarrow/tests/parquet/conftest.py: ########## @@ -53,23 +53,22 @@ def s3_bucket(s3_server): @pytest.fixture def s3_example_s3fs(s3_server, s3_bucket): Review Comment: I think the point of having both is because we want to test both our own S3FileSystem as also the external s3fs one in some cases ? Of course now with how you rewrote it, it's duplicated, but don't we have to keep the s3fs one for _some_ tests? -- 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]
