Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-37390: [Python] Expose StreamDecoder API to pyarrow #37371

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kdkavanagh
Copy link

@kdkavanagh kdkavanagh commented Aug 24, 2023

Rationale for this change

The stream decoder API allows for more fine-grained control when reading arrow streams that are being concurrently written to. Reads of the stream can be handled manually, with bytes offered incrementally to the stream decoder API.

Are these changes tested?

Yes

@kou
Copy link
Member

kou commented Aug 25, 2023

This is not a MINOR change: https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes

Could you open a new issue for this?

@kdkavanagh kdkavanagh changed the title MINOR: [Python] Expose StreamDecoder API to pyarrow GH-37390: [Python] Expose StreamDecoder API to pyarrow Aug 25, 2023
@kdkavanagh
Copy link
Author

kdkavanagh commented Aug 25, 2023

Done: #37390

@github-actions
Copy link

⚠️ GitHub issue #37390 has been automatically assigned in GitHub to PR creator.

@AlenkaF
Copy link
Member

AlenkaF commented Aug 28, 2023

Thank you for the contribution @kdkavanagh!
There are some corrections needed to have a green CI.

INFO:archery:Running Python docstring linters

pyarrow.ipc.StreamDecoder
PR01: Parameters {'memory_pool'} not documented

Sign up for free to 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.

[Python] Expose StreamDecoder to pyarrow python API
3 participants