milesgranger commented on code in PR #14011:
URL: https://github.com/apache/arrow/pull/14011#discussion_r959516502
##########
python/pyarrow/parquet/core.py:
##########
@@ -1803,6 +1803,9 @@ def __init__(self, path_or_paths, filesystem=None,
schema=None,
raise NotImplementedError("split_row_groups not yet implemented")
if filters is not None:
+ import pyarrow.dataset as ds
+ if isinstance(filters, ds.Expression):
Review Comment:
I _think_ this might be unsafe, if dataset wasn't built. An alternative is
to remove and just let it fail down in _check_filters, which will complain
about `Expression` not having `len`.
--
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]
