You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/08/05 09:28:10 UTC

[GitHub] [arrow] jorisvandenbossche commented on a diff in pull request #13629: ARROW-16719: [Python] Add path/URI /+ filesystem handling to parquet.read_metadata

jorisvandenbossche commented on code in PR #13629:
URL: https://github.com/apache/arrow/pull/13629#discussion_r938631912


##########
python/pyarrow/tests/parquet/test_metadata.py:
##########
@@ -18,12 +18,15 @@
 import datetime
 import decimal
 from collections import OrderedDict
+import os
 
 import numpy as np
 import pytest
 
 import pyarrow as pa
 from pyarrow.tests.parquet.common import _check_roundtrip, make_sample_file
+from pyarrow.filesystem import LocalFileSystem, FileSystem

Review Comment:
   Sorry for only noticing this late, but we should import here from `pyarrow.fs` to use the new filesystems in the tests (`pyarrow.filesystem` is deprecated)



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