fqaiser94 opened a new pull request, #714: URL: https://github.com/apache/iceberg-rust/pull/714
# Not Ready For Reviews I don't recommend reviewing this formally just yet as I plan to break this up into multiple PRs to make it easier to review and I also have some minor TODOs to complete but otherwise, it should be more-or-less functionally complete and at feature parity with the Java implementation (I've even verified the outputs are bit-wise identical). # Summary Adds [Puffin](https://iceberg.apache.org/puffin-spec/ ) reader and writer implementations. # Out of scope - LZ4 compression/decompression - The Java library does not currently support this either. As the Java implementations are the reference implementations in the Iceberg ecosystem, I would like implement LZ4 support on the Java side first before implementing on the Rust side. At that point, we can also easily check the Java and Rust generated outputs are bit-wise identical. # Remaining Work - [ ] Finish TODOs - [ ] Review `pub`/`pub(crate)`/`private` visibility - [ ] Figure out if we need to integrate with opendal or something - [ ] Split into multiple PRs -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
