jorisvandenbossche commented on code in PR #14717:
URL: https://github.com/apache/arrow/pull/14717#discussion_r1040728222
##########
python/pyarrow/tests/parquet/test_parquet_file.py:
##########
@@ -177,8 +178,12 @@ def
test_parquet_file_pass_directory_instead_of_file(tempdir):
path = tempdir / 'directory'
os.mkdir(str(path))
- with pytest.raises(IOError, match="Expected file path"):
+ msg = f"Cannot open for reading: path '{str(path)}' is a directory"
+ with pytest.raises(IOError) as exc:
pq.ParquetFile(path)
+ if exc.errisinstance(PermissionError) and sys.platform == 'win32':
+ return # Windows CI can get a PermissionError here.
Review Comment:
> is memory-mapping activated?
That's optional, with the default of False, so it shouldn't be used in this
case
--
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]
