mqy commented on pull request #9137: URL: https://github.com/apache/arrow/pull/9137#issuecomment-757438089
> > Perhaps you missed `RecordBatchBuilder::add_compression()`? > > I missed it at the dictionary write, but not when writing a plain recordbatch; so it's not that. If I hadn't written the compression details completely, the C++ implementation wouldn't have known that the message is compressed or with LZ4. > I'll debug when I have sufficient time @nevi-me I pulled your branch locally, found `add_compression` in `writer.rs`. I'm sorry that I failed to search `add_compression` from `https://github.com/apache/arrow/pull/9137/fies` because `writer.rs` is not loaded. I'm installing `pyarrow`, after run test `test_write_file_v5_compressed`. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
