You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "AlenkaF (via GitHub)" <gi...@apache.org> on 2023/09/07 04:33:10 UTC

[GitHub] [arrow] AlenkaF commented on a diff in pull request #37558: GH-37555: [Python] Update test_get_file_info_with_selector

AlenkaF commented on code in PR #37558:
URL: https://github.com/apache/arrow/pull/37558#discussion_r1318071012


##########
python/pyarrow/tests/test_fs.py:
##########
@@ -703,6 +703,10 @@ def test_get_file_info_with_selector(fs, pathfn):
             # on the s3fs/fsspec version combo, it includes the base_dir
             # (https://github.com/dask/s3fs/issues/393)
             assert (len(infos) == 4) or (len(infos) == 5)
+        elif fs.type_name in ["py::fsspec+file", 'py::fsspec+memory']:
+            # fsspec also lists root dir
+            # GH-37555

Review Comment:
   If I understand correctly, you think we should work around it in a way that we do not include the root dir in the result?



-- 
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: github-unsubscribe@arrow.apache.org

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