patrickpai commented on pull request #7789: URL: https://github.com/apache/arrow/pull/7789#issuecomment-660280975
@pitrou I can implement your heuristic. What I can do is in the `SerializedPageReader` constructor, start reading the first page, detect if we need to use hadoop lz4 or non-hadoop lz4, and then initialize `std::unique_ptr<::arrow::util::Codec> decompressor_;`. Note that this will complicate the `SerializedPageReader` class. ---------------------------------------------------------------- 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
