jorisvandenbossche commented on code in PR #33969: URL: https://github.com/apache/arrow/pull/33969#discussion_r1099817496
########## python/pyarrow/tests/test_dataset.py: ########## @@ -5084,10 +5084,8 @@ def test_dataset_partition_with_slash(tmpdir): read_table = ds.dataset( source=path, format='ipc', - partitioning='hive', - schema=pa.schema([pa.field("exp_id", pa.int32()), - pa.field("exp_meta", pa.utf8())]) - ).to_table().combine_chunks() + schema=dt_table.schema, Review Comment: > The issue is with the existing test case. When the schema is provided that case failed Can you show the error that you got for this test before you made the changes to `_ensure_partitioning`? (and to be clear, the test also fails without the modifications to the test code? As those changes are only stylistic?) -- 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