Skip to content

ARROW-12428: [Python] Expose pre_buffer in pyarrow.parquet - #10074

Closed
lidavidm wants to merge 4 commits into
apache:masterfrom
lidavidm:arrow-12428
Closed

ARROW-12428: [Python] Expose pre_buffer in pyarrow.parquet#10074
lidavidm wants to merge 4 commits into
apache:masterfrom
lidavidm:arrow-12428

Conversation

@lidavidm

Copy link
Copy Markdown
Member

This allows using the new option without going through the datasets API, such as to read a single file. A simple benchmark is in the JIRA. This helps close the gap between PyArrow and fsspec read performance, as fsspec performs readahead by default.

@github-actions

Copy link
Copy Markdown

@lidavidm

Copy link
Copy Markdown
MemberAuthor

The pre-buffer/Parquet logic has a bug with empty files, which I will need to fix.

@lidavidmlidavidm changed the title ARROW-12428: [Python] Expose pre_buffer in pyarrow.parquetARROW-12428: [C++][Python] Expose pre_buffer in pyarrow.parquetApr 16, 2021
Comment threadpython/pyarrow/parquet.py Outdated
Comment threadpython/pyarrow/parquet.py Outdated
@jorisvandenbossche

Copy link
Copy Markdown
Member

Thanks @lidavidm ! (I know it goes against the point of an RC, but it would be nice if we could still get this into 4.0 ..)

@lidavidmlidavidm changed the title ARROW-12428: [C++][Python] Expose pre_buffer in pyarrow.parquetARROW-12428: [Python] Expose pre_buffer in pyarrow.parquetMay 4, 2021
@lidavidm

Copy link
Copy Markdown
MemberAuthor

I've rebased this, and tests pass. (Sorry for the long delay here.)

Comment threadpython/pyarrow/parquet.py Outdated
Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Comment threadpython/pyarrow/parquet.py Outdated
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@lidavidm@jorisvandenbossche