AlenkaF commented on code in PR #47144:
URL: https://github.com/apache/arrow/pull/47144#discussion_r2218435862
##########
python/pyarrow/parquet/core.py:
##########
@@ -1881,7 +1881,26 @@ def read_table(source, *, columns=None, use_threads=True,
filesystem, path = _resolve_filesystem_and_path(source, filesystem)
if filesystem is not None:
source = filesystem.open_input_file(path)
- # TODO test that source is not a directory or a list
+ if isinstance(source, list):
+ raise ValueError(
+ "Cannot read list of files with ParquetFile when
pyarrow.dataset "
+ "module is not available. Install pyarrow with dataset
support."
+ )
+
+ if isinstance(source, (str, os.PathLike)):
+ source_path = _stringify_path(source)
Review Comment:
Could we use `path` from line 1881 where we call
`_resolve_filesystem_and_path` to make this a bit simpler?
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
