JingsongLi commented on PR #727: URL: https://github.com/apache/paimon-rust/pull/727#issuecomment-5350373063
This has been overtaken by current `main`: #731 now contains the same Java-compatible LZ4 block decoder (including the outer varint/header cross-check) and already turns `test_java_compat_int_lz4` into a passing compatibility test. I also verified locally that merging this head into current `main` conflicts in both `crates/paimon/src/btree/sst_file.rs` and `crates/paimon/src/btree/tests.rs`. Please rebase and drop the duplicated decoder/test changes (or close this PR if no independent change remains). The additional cross-codec equivalence assertion may still be worth preserving as a small follow-up, but the core fix is already present on `main`. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
