yihua commented on PR #836: URL: https://github.com/apache/hudi-rs/pull/836#issuecomment-6476497644
Thanks for the fix! One small follow-up, pushed as 58017acd: since the block header already carries `on_disk_data_size_with_header`, the reader now slices the compressed payload at that value directly, the same way the Java reader's `unpack()` does, instead of deriving it back from the checksum chunk count. That drops `checksum_bytes_count` and its header-only test; the GZIP boundary test covers the same case through `HFileBlock::parse`. I updated the description to match. -- 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]
