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

feat(rust): add BinaryView to parquet writer/reader. #13489

Merged
merged 14 commits into from
Jan 10, 2024
Merged

feat(rust): add BinaryView to parquet writer/reader. #13489

merged 14 commits into from
Jan 10, 2024

Conversation

ritchie46
Copy link
Member

@ritchie46 ritchie46 commented Jan 6, 2024

Adds BinaryView/StringView to parquet writer and reader. Also found some small things that will improve parquet performance. Will follow up on that one in future PRs.

This temporary increases code bloat. But later we can remove the (Large)Utf8/Binary branches.

@github-actions github-actions bot added enhancement New feature or an improvement of an existing feature rust Related to Rust Polars labels Jan 6, 2024
@ritchie46 ritchie46 force-pushed the binview branch 2 times, most recently from 0145771 to 79a1782 Compare January 9, 2024 11:11
@ritchie46 ritchie46 merged commit 505ec71 into main Jan 10, 2024
19 checks passed
@ritchie46 ritchie46 deleted the binview branch January 10, 2024 15:35
@c-peters c-peters added the accepted Ready for implementation label Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted Ready for implementation enhancement New feature or an improvement of an existing feature rust Related to Rust Polars
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants