-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
IpcWriter with compression #1769
IpcWriter with compression #1769
Conversation
That would be great. :) |
Already reviewed the Rust code. Looks good! 👍 |
Now parquet Compression is re-exported as Compression, it may occur conflicts with others. Maybe is it better to rename like ParquetCompression? polars/polars/polars-io/src/parquet.rs Line 194 in 4e6dea0
|
Yes, I think that's a good idea. Maybe even create our own enums, so that we don't have an API break, when arrow2 adds an option. |
Thanks a lot! |
I implemented with_compression in IPC Writer and its test related to #1539.
If necessary, I would like to expose the Python API.