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


##########
python/pyarrow/parquet/core.py:
##########
@@ -2961,129 +1949,77 @@ def partitioning(self):
 """
 
 
-def read_table(source, *, columns=None, use_threads=True, metadata=None,
+def read_table(source, *, columns=None, use_threads=True,
                schema=None, use_pandas_metadata=False, read_dictionary=None,
                memory_map=False, buffer_size=0, partitioning="hive",
-               filesystem=None, filters=None, use_legacy_dataset=False,

Review Comment:
   Specifying `use_legacy_dataset=False` doesn't break in the current code due 
to **kwargs kept in `ParquetDataset` from `_ParquetDatasetV2`. I would 
personally keep it as is as it gets ignored currently and would add a 
deprecation warning and a test for it.
   



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

Reply via email to